Implement NOT VALID for Postgres databases

[coenvk]

Environment

Liquibase Version: 4.7.1

Liquibase Integration & Version: CLI

Liquibase Extension(s) & Version:

Database Vendor & Version: PostgreSQL 9.1

Operating System Type & Version: Microsoft Windows 11 Home

Pull Request Type

• Bug fix (non-breaking change which fixes an issue.)
• Enhancement/New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Description

PostgreSQL has implemented a feature, since version 9.1, that ensures that constraints are only checked for new or modified rows, but not for existing data. When defining a constraint, NOT VALID can be added at the end to make use of this feature. A similar feature is available in Oracle under the name of ENABLE NOVALIDATE. Currently, NOT VALID is only allowed for foreign key and check constraints.

Steps To Reproduce

<changeSet id="addfkey" author="coenvk">
    <addForeignKeyConstraint baseTableName="account" baseColumnNames="customer_id" referencedTableName="customer" referencedColumnNames="id" constraintName="account_customer_id_fkey" validate="false"/>
</changeSet>

liquibase updateSQL

Actual Behavior

ALTER TABLE public.account ADD CONSTRAINT account_customer_id_fkey FOREIGN KEY (customer_id) REFERENCES public.customer (id) NOT VALID;

Expected/Desired Behavior

Executing the addForeignKeyConstraint migration with validate="false" on a Postgres database will now append "NOT VALID" add the end of the SQL and ensures that existing data will not validated by the foreign key constraint.

[nvoxland]

Code Review and test results:

Change makes sense, thanks!

Things to be aware of

• I created Added check for validate=false on addForeignKeyConstraint for Postgresql liquibase-test-harness#240 as an automated test for it

Things to worry about

[github-actions]

Unit Test Results

  4 548 files  ±0    4 548 suites  ±0   30m 6s ⏱️ - 2m 19s
  4 508 tests ±0    4 294 ✔️ ±0     214 💤 ±0  0 ❌ ±0 
53 376 runs  ±0  48 368 ✔️ ±0  5 008 💤 ±0  0 ❌ ±0 

Results for commit dee23ba. ± Comparison against base commit 4e9b0d5.

[XDelphiGrl]

• Fix is straight-forward and localized to the Postgres databases 9.1 or higher.
• Test added to the test harness.
  • PR

APPROVED

Passing Functional Tests