[I] REST Catalog alterTable silently reports success when the server returns a non-table/column 404 [paimon]

["LuciferYang (via GitHub)" <gi...@apache.org>]

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: issues-unsubscribe@paimon.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] REST Catalog alterTable silently reports success when the server returns a non-table/column 404 [paimon]

["JingsongLi (via GitHub)" <gi...@apache.org>]

JingsongLi closed issue #10267: REST Catalog alterTable silently reports success when the server returns a non-table/column 404
URL: https://github.com/apache/paimon/issues/10267


-- 
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: issues-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org