wangxfholly commented on PR #39540:
URL: https://github.com/apache/shardingsphere/pull/39540#issuecomment-5459497241
Hi @zhang-arvin @terrymanu,
I have a local candidate fix for #39084 and would like to preserve strict
fail-fast semantics:
- Validate that all expected tables are present in the loaded metadata.
- Throw SQLException when table metadata is missing or incomplete.
- Propagate dialect-loader SQLExceptions instead of silently falling back.
- Cancel remaining metadata-loading tasks after a failure.
- Convert thread interruption into SQLException instead of returning
partial metadata.
This is intentionally stricter than the current WARN-and-continue approach
in this PR. Could you confirm whether this fail-fast behavior is acceptable for
#39084? If so, I can adapt the change to the latest master and either
contribute it to this PR or open a separate PR as requested.
--
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]
