Search before asking
Paimon version
master (1.5-SNAPSHOT)
Compute Engine
Any engine using the REST Catalog.
Minimal reproduce step
- Use a REST catalog.
- 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?
Search before asking
Paimon version
master (1.5-SNAPSHOT)
Compute Engine
Any engine using the REST Catalog.
Minimal reproduce step
alterTable(identifier, changes, ignoreIfNotExists = false)in a situation where the REST server answers with a 404 whose resource type is neitherTABLEnorCOLUMN, for example aDATABASEnot-found because the table's database was concurrently dropped.What doesn't meet your expectations?
alterTablereturns normally and reports success even though the ALTER did not apply. TheNoSuchResourceExceptionhandler only covers theTABLEandCOLUMNresource 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
createBranchalready rethrows in the analogous default branch of the same kind of dispatch.alterTablewas the only one that let an unmapped 404 fall through to silent success.Are you willing to submit a PR?