liquibase-redshift icon indicating copy to clipboard operation
liquibase-redshift copied to clipboard

AutoIncrement should convert to { INT | BIGINT } IDENTITY, instead of { SERIAL | BIGSERIAL }

Open DJFarr opened this issue 7 months ago • 0 comments

Hi,

I've seen this issue posted on the internet a few times, and even one issue in the liquibase repo which as been open for years (https://github.com/liquibase/liquibase/issues/3505).

As stated, an INT or BIGINT column set to AutoIncrement: true in redshift should actually generate an IDENTITY column, as its the alternative to SEQUENCES.

I have connected to a new redshift instance, and put the driver and this extension in my liquibase/lib/ folder, and when i run liquibase update-sql the generated SQL contains SERIALs.

image

image

image

Doc: https://docs.aws.amazon.com/redshift/latest/dg/r_CREATE_TABLE_NEW.html Examples: https://docs.aws.amazon.com/redshift/latest/dg/r_CREATE_TABLE_examples.html

I felt like this was a better place for this issue to sit rather than the core repo. I'm new to liquibase and this repo so I'm not confident enough yet to make my own PR.

DJFarr avatar Jul 11 '24 06:07 DJFarr