LuciferYang opened a new issue, #10267:
URL: https://github.com/apache/paimon/issues/10267

   ### Search before asking
   
   - [X] I searched in the [issues](https://github.com/apache/paimon/issues) 
and found no similar issues.
   
   ### Paimon version
   
   master (1.5-SNAPSHOT)
   
   ### Compute Engine
   
   Any engine using the REST Catalog.
   
   ### Minimal reproduce step
   
   1. Use a REST catalog.
   2. Call `alterTable(identifier, changes, ignoreIfNotExists = false)` in a 
situation where the REST server answers with a 404 whose resource type is 
neither `TABLE` nor `COLUMN`, for example a `DATABASE` not-found because the 
table's database was concurrently dropped.
   
   ### What doesn't meet your expectations?
   
   `alterTable` returns normally and reports success even though the ALTER did 
not apply. The `NoSuchResourceException` handler only covers the `TABLE` and 
`COLUMN` resource types; for any other type it falls through the if/else-if 
chain and the method returns without error, so a failed alter is silently 
swallowed.
   
   ### Anything else?
   
   The sibling `createBranch` already rethrows in the analogous default branch 
of the same kind of dispatch. `alterTable` was the only one that let an 
unmapped 404 fall through to silent success.
   
   ### Are you willing to submit a PR?
   
   - [X] I'm willing to submit a PR!
   


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