rishi-rana commented on PR #39882:
URL: https://github.com/apache/shardingsphere/pull/39882#issuecomment-6101478516

   Thanks, both confirmed and addressed in 553a0f376fb.
   
   1. `ContextManager.java:307` — the speculative route is now removed only 
when it wasn't already registered before this call. A table that already had a 
persisted route is left for `dropTables()` to clean up itself, so its own 
`isNeedRefresh` check sees the route still present and persists the SINGLE 
config change correctly.
   2. `StandardDatabaseProxyConnector.java:296-299` — each table's schema is 
now resolved from its own `SimpleTableSegment.getOwner()`, falling back to the 
statement's default schema only when a table isn't schema-qualified.
   
   Verification: 923 tests pass across the full `mode/core` and 
`proxy/backend/core` modules; `spotless:check checkstyle:check -Pcheck` clean. 
Also re-verified `checkstyle:check -Pcheck` against a worktree merging this 
branch onto current `master`, given the last round's surprise.
   
   3. Still awaiting your sign-off on the support contract.
   


-- 
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]

Reply via email to