hesam-oxe commented on PR #11077:
URL: https://github.com/apache/seatunnel/pull/11077#issuecomment-6131758419
Follow-up on the new head `8b383c04e`: fixed one failing unit test.
**Finding:**
`MultiTableSinkWriterTest.testSharedDestinationWithDivergentSchemasFailsFast`
failed on `assertTrue(message.contains("dest.db.shared"))`. I verified it fails
identically on the pre-sync commit, so it is a pre-existing test/implementation
mismatch, not a sync regression: `validateSharedDestinationSchemas`
intentionally redacts the destination via `logSafeDestinationId` (identifiers
may carry endpoints/credentials), so the raw path can never appear in the
message.
**Fix (`[Test][API]`, test-only):** the test now asserts the redaction
contract — both source tables plus the `destination-hash=` marker must be
present, and the raw destination path must be absent.
**Local verification:** all 51 `multitablesink` tests green,
`spotless:check` green.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
