LuciferYang opened a new pull request, #10268: URL: https://github.com/apache/paimon/pull/10268
### Purpose `RESTCatalog.alterTable` handles a `NoSuchResourceException` by mapping the `TABLE` and `COLUMN` resource types to the right behavior, but with `ignoreIfNotExists = false` any other resource type falls through the if/else-if chain and the method returns normally, reporting success on an alter that did not apply. This happens when the server returns a non-table/column 404, for example a `DATABASE` not-found because the table's database was concurrently dropped. This rethrows the exception for the unmapped resource types, so the failure reaches the caller. The sibling `createBranch` already rethrows in the analogous default branch of the same dispatch. The `ignoreIfNotExists = true` path is unchanged and still swallows every 404. This closes #10267. ### Tests - `MockRESTCatalogTest` gains a case that injects a non-table/column 404 into `alterTable` with `ignoreIfNotExists = false` and asserts the exception propagates. Without the fix the method returned normally, so the assertion failed. ### API and Format No. ### Documentation No. -- 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]
