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

   ### What changes were proposed in this pull request?
   
   Give eleven built-in column/index TableChange classes readable descriptions 
and format conflicting rename changes as a comma-separated list. Add a default 
connector hook for an unsupported sort-order explanation, and use it in the 
JDBC create guard before database access.
   
   ### Why are the changes needed?
   
   JDBC rejections expose Java class names/object hashes and hide 
connector-specific recovery advice.
   
   Fixes #14497
   
   ### Does this PR introduce _any_ user-facing change?
   
   Only string representations and rejection messages change. Existing 
IllegalArgumentException/HTTP validation behavior and rejection before DDL are 
retained. The default hook works for existing connectors; connectors may 
override it without changing capability support.
   
   ### How was this patch tested?
   
   - Full API and JDBC common non-Docker unit suites: 323 and 128 tests, no 
failures.
   - Added assertions for all eleven descriptions, conflicting rename messages 
and sort-order rejection through the catalog entry point.
   - Reverting the renderers makes both API and catalog regressions fail; 
restoring the patch passes both full suites again.
   - Spotless, RAT and independent fresh-context review passed.
   - No Docker/live connector or REST end-to-end test was run; changed 
rejection paths run before database access.
   


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