Skip to content

[split 9/22] lightweight: quote offset-table identifiers per dotted component, offset storage hardening - #1364

Open
minguyen9988 wants to merge 1 commit into
2.10.0from
omniwatcher/split-09-lw-offset-storage
Open

[split 9/22] lightweight: quote offset-table identifiers per dotted component, offset storage hardening#1364
minguyen9988 wants to merge 1 commit into
2.10.0from
omniwatcher/split-09-lw-offset-storage

Conversation

@minguyen9988

Copy link
Copy Markdown
Collaborator
  • DebeziumOffsetStorage quoted the database-qualified offset table name as ONE identifier - every start against a db.table configured name died with UNKNOWN_TABLE. Identifiers are now quoted per dotted component. DebeziumOffsetStorageQuotingTest pins it.
  • DebeziumJdbcStorageOperations: parameterized statements and consistent quoting on the offset/history storage path (DebeziumJdbcStorageOperationsTest).
  • ReplicationStatusSingleton simplification; REST API response-shape fix in DebeziumEmbeddedRestApi.

Part of the split of #1353 into independently mergeable sub-PRs (each <= 10 files), so the 2.10.0 branch can absorb the fixes incrementally.

Split out of #1353, which this series replaces. Each sub-PR is <= 10 files; the union of all 22 reproduces the #1353 tree exactly (verified by tree SHA).

…set storage hardening

- DebeziumOffsetStorage quoted the database-qualified offset table name as ONE identifier - every start against a db.table configured name died with UNKNOWN_TABLE. Identifiers are now quoted per dotted component. DebeziumOffsetStorageQuotingTest pins it.
- DebeziumJdbcStorageOperations: parameterized statements and consistent quoting on the offset/history storage path (DebeziumJdbcStorageOperationsTest).
- ReplicationStatusSingleton simplification; REST API response-shape fix in DebeziumEmbeddedRestApi.

Part of the split of #1353 into independently mergeable sub-PRs (each <= 10 files), so the 2.10.0 branch can absorb the fixes incrementally.
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