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

Use default value for the dbaas lc_* database fields #260

Merged

Conversation

Gogen120
Copy link
Contributor

Use default value instead of DiffSuppressFunc because of different behavior of the PostgreSQL and PostgreSQL-1C databases

@Gogen120 Gogen120 force-pushed the use_default_value_for_lc_database_fields branch from 7020969 to ca28495 Compare April 18, 2024 08:08
@TawR1024
Copy link
Collaborator

@Gogen120, please update this PR

@Gogen120 Gogen120 force-pushed the use_default_value_for_lc_database_fields branch from ca28495 to 6f9ee81 Compare April 23, 2024 13:44
Use default value instead of DiffSuppressFunc because of different
behavior of the PostgreSQL and PostgreSQL-1C databases
@Gogen120 Gogen120 force-pushed the use_default_value_for_lc_database_fields branch from b40900a to 758fe54 Compare May 16, 2024 05:55
Copy link
Collaborator

@TawR1024 TawR1024 left a comment

Choose a reason for hiding this comment

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

LGTM

@kolsean kolsean merged commit 2a58a79 into selectel:master May 16, 2024
2 checks passed
@Gogen120 Gogen120 deleted the use_default_value_for_lc_database_fields branch September 20, 2024 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants