[DBIx-Class-Devel] [dbsrgits/sql-translator] add support for covering index (INCLUDE) on PG producer (#131)

[Veesh Goldman]

This adds support for covering indices (i.e. the "INCLUDE" clause on `create
index`) for the Postgres producer.

the intended interface is as follows

    $table->add_index(
       name => 'covering', fields => [ 'foo' ], 
       options => { include => [ 'bar', 'baz' ] }
    );

In the current implementation it silently skips the clause if PG is under
version 10, b/c it was only added in 11 (as far as I remember). I'm not quite
sure if it should be that way, or if it should die.

You can view, comment on, or merge this pull request online at:

  https://github.com/dbsrgits/sql-translator/pull/131

-- Commit Summary --

  * add support for covering index (INCLUDE) on PG producer

-- File Changes --

    M lib/SQL/Translator/Producer/PostgreSQL.pm (16)
    M t/47postgres-producer.t (8)

-- Patch Links --

https://github.com/dbsrgits/sql-translator/pull/131.patch
https://github.com/dbsrgits/sql-translator/pull/131.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/dbsrgits/sql-translator/pull/131
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.scsys.co.uk/pipermail/dbix-class-devel/attachments/20210202/923edf16/attachment.htm>