Skip to content

Conversation

@yureiblack
Copy link

Summary of Fixes

1. Removed Unconditional Early Return Blocking Validation

  • Removed the if (true) return false; statement that was preventing
    the #checkRequiredProperties() method from executing its validation logic.
  • This ensures that required property checks are no longer skipped.

2. Added Missing break in INCRVACUUM Switch Case

  • Added the missing break statement to prevent fall-through into the
    default case.
  • This fixes potential logic issues when the INCRVACUUM property is used.

Result

These changes ensure that all required properties —
RECURSIVE_TRIGGERS, TEMPSTORE_FILE, TEMPSTORE_MEM, AUTOVACUUM, and INCRVACUUM
are now properly validated and applied within the test scripts.

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.

1 participant