Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #992: Review liquibase scripts #993

Merged
merged 8 commits into from
Sep 4, 2023

Conversation

jandusil
Copy link
Contributor

@jandusil jandusil commented Sep 1, 2023

No description provided.

@jandusil jandusil requested a review from zcgandcomp September 1, 2023 14:02
@jandusil jandusil self-assigned this Sep 1, 2023
@jandusil jandusil linked an issue Sep 1, 2023 that may be closed by this pull request
@jandusil jandusil changed the title Issues/992 review liquibase scripts Review liquibase scripts Sep 1, 2023
@jandusil jandusil changed the title Review liquibase scripts Fix #992: Review liquibase scripts Sep 1, 2023
Copy link
Member

@zcgandcomp zcgandcomp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please keep current name of indexes

- Revert index name changes in audit_log
@jandusil jandusil requested a review from zcgandcomp September 4, 2023 11:16
@@ -70,7 +75,7 @@
</not>
</preConditions>
<comment>Create a new sequence pa_recovery_puk_seq</comment>
<createSequence sequenceName="pa_recovery_puk_seq" cacheSize="20" />
<createSequence sequenceName="pa_recovery_puk_seq" startValue="1" incrementBy="1" cacheSize="20" type="bigint"/>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Different type then id in the table

@@ -80,7 +85,8 @@
</not>
</preConditions>
<comment>Create a new sequence pa_recovery_config_seq</comment>
<createSequence sequenceName="pa_recovery_config_seq" cacheSize="20" />
<createSequence sequenceName="pa_recovery_config_seq" startValue="1" incrementBy="1" cacheSize="20"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Different type then id in the table

Copy link
Member

@zcgandcomp zcgandcomp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In a nutshell only new attributes for sequence are added. Otherwise it is kept as in the original SQL files.

LB is runnable on all supported DB platforms.

@jandusil jandusil merged commit bd40dcf into develop Sep 4, 2023
5 checks passed
@jandusil jandusil deleted the issues/992-review-liquibase-scripts branch September 4, 2023 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Review liquibase scripts
2 participants