lasdf1234 commented on PR #12999: URL: https://github.com/apache/gravitino/pull/12999#issuecomment-5597300489
@yuqi1129 Thanks for the focused review. Addressed all three P2s on `ExceptionMessages` in 95c8c43bb: 1. Prefer shallow non-blank context (skip transparent wrappers; append deeper reason only when needed) so nested wraps keep intermediate diagnostics 2. Detect multi-node cause cycles via identity-based visited set 3. Ignore blank/whitespace-only messages when selecting candidates Unit tests updated/added in `TestExceptionMessages` (nested wrap, blank messages, 2-/3-node cycles). -- 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]
