yuqi1129 opened a new pull request, #14514:
URL: https://github.com/apache/gravitino/pull/14514

   ### What changes were proposed in this pull request?
   
   Backport the existing main-branch ConnectionFailedException-to-HTTP-502 
fallback into the branch-1.3 base REST exception handler. Add schema 
LIST/LOAD/CREATE/DROP coverage and non-connection status controls.
   
   ### Why are the changes needed?
   
   On branch-1.3, schema connection failures fall through to generic HTTP 500 
while table calls report 502. Main already contains the mapping introduced with 
PR #11959; this change ports only the small fallback and its regression 
coverage.
   
   Fixes #14503
   
   ### Does this PR introduce _any_ user-facing change?
   
   Schema source-connection failures now return 502 with 
ConnectionFailedException. Existing validation, missing-schema, permission and 
unexpected-error statuses remain 400, 404, 403 and 500.
   
   ### How was this patch tested?
   
   All 473 branch-1.3 server module tests passed with -PskipITs. The added 
schema connection regression failed with 500 before the fix and passes for 
LIST/LOAD/CREATE/DROP afterward. Module Spotless passed.
   


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