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

   ### What changes were proposed in this pull request?
   
   Recognize Doris `Unknown table` errors with quoted or unquoted names and 
with or without a cluster-qualified schema. Remove repeated 
`errCode/detailMessage` prefixes before converting the exception. Add unit and 
integration regression tests.
   
   ### Why are the changes needed?
   
   Doris currently treats a missing table on DROP as a generic runtime error, 
so the API returns HTTP 500 instead of `dropped:false`.
   
   Fix: #13343
   
   ### Does this PR introduce _any_ user-facing change?
   
   Yes. Dropping a missing Doris table returns HTTP 200 with `dropped:false`. 
Doris error messages no longer repeat the same backend prefix. No API or 
property keys change.
   
   ### How was this patch tested?
   
   - `./gradlew spotlessApply`
   - `./gradlew :catalogs:catalog-jdbc-doris:test -PskipITs 
-PskipDockerTests=true -PskipWeb=true` (61 tests passed)
   - `./gradlew :catalogs:catalog-jdbc-doris:test --tests 
org.apache.gravitino.catalog.doris.integration.test.CatalogDorisIT.testDropMissingTableReturnsFalse
 -PskipDockerTests=false -PdorisMultiVersionTest -PskipWeb=true` (passed with 
Doris container)
   


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