cacheSize of createSequence is ignored

Environment

Liquibase Version: 4.4.3

Liquibase Integration & Version: quarkus 2.3.0.Final

Liquibase Extension(s) & Version: liquibase-core only

Database Vendor & Version: MariaDB 10.6.4

Operating System Type & Version: Windows 10

Description

The Attribute cacheSize of createSequence seems to be ignored.

Steps To Reproduce

Run the following changesets:

    <changeSet id="working" author="mickroll">
        <createSequence sequenceName="SEQ_WORKING"
                            startValue="10000000"
                            incrementBy="1" />
        <sql dbms="mysql,mariadb">ALTER SEQUENCE SEQ_WORKING CACHE=5;</sql>
    </changeSet>
    <changeSet id="attribute" author="mickroll">
        <createSequence sequenceName="SEQ_SET"
                            startValue="10000000"
                            incrementBy="1"
                            cacheSize="5" />
    </changeSet>
    <changeSet id="default" author="mickroll">
        <createSequence sequenceName="SEQ_DEFAULT"
                            startValue="10000000"
                            incrementBy="1" />
    </changeSet>

• changeset working shows the wanted end result
• changeset attribute shows usage of createSequence cacheSize (which will be ignored)
• changeset default shows the default value of cacheSize (as defined by the database, i guess)

Now run the following SQL on the database to extract the results (or view the created sequences in the db client of our choice):

SELECT (SELECT cache_size FROM SEQ_WORKING) AS "working",
       (SELECT cache_size FROM SEQ_SET) AS "set with attribute",
       (SELECT cache_size FROM SEQ_DEFAULT) AS "default";

Actual Behavior

• working = 5 (as expected)
• set with attribute = 1000 (bug)
• default = 1000 (shows default value)

Expected/Desired Behavior

• set with attribute = 5 (as defined by the attribute cacheSize of createSequence tag)

Screenshots (if appropriate)

Additional Context

Doesn’t work, either: <alterSequence sequenceName="SEQ_SET" cacheSize="5"/>

┆Issue is synchronized with this Jira Story by Unito

[nvoxland]

It may be better to have more function checks like that vs instanceof. The trouble is that there is sort of an unlimited variation in what features are supported and not supported across the databases and also we hit variations in the SQL to generate depending on the database as well.

We’re looking at some refactoring to make that all work better, but it’s a medium-term project. In the meantime, I opened a PR that handles it the current way.

[mickroll]

Thank you!