Skip to content

fix(sql): close non-transactional execution connections - #960

Open
bercianor wants to merge 3 commits into
masterfrom
fix/sql-connection-lifecycle
Open

bercianor wants to merge 3 commits into
masterfrom
fix/sql-connection-lifecycle

Conversation

@bercianor

Copy link
Copy Markdown
Contributor

Summary

  • Close the JDBC connection injected for non-transactional SQL changes after apply or rollback.
  • Make container-backed SQL test setup exception-safe and use the Informix-aware datasource helper.
  • Add regression coverage for apply, rollback, callback failure, and datasource setup failure.

Verification

  • :core:target-systems:flamingock-sql-targetsystem:test with Java 17.0.8-graal
  • :community:flamingock-sql-auditstore:test --tests "*SqlAuditStoreTest*" -Psql.test.dialects=informix --no-parallel with Java 17.0.8-graal
  • Nine-dialect SQL audit-store matrix with Java 17.0.8-graal

Scope

This PR contains only the SQL connection-lifecycle and Informix test-lifecycle bugfix. The SQL journal payload feature remains separate uncommitted WIP.

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