jackylee-ch opened a new pull request, #830: URL: https://github.com/apache/paimon-rust/pull/830
The five `resourceType` values this server emits are lowercase, but Java's `ErrorResponse` declares them as `DATABASE` / `TABLE`, and a Java client dispatches on the field: `RESTCatalog#alterTable` raises `TableNotExistException` only when it equals `RESOURCE_TYPE_TABLE`, comparing char by char through `StringUtils#equals`. With `table` neither branch matches, so `alterTable(ident, changes, false)` on a missing table silently returns instead of throwing. `createBranch` rethrows in the same position, so the symptom varies by call. They are now the uppercase constants. No effect on the Rust client, which keys off the status code alone. Left alone deliberately: the two column errors still answer 400 with `column:<table>`. Making them Java's 404 plus `COLUMN` needs the Rust client changed too, which would otherwise read a missing column as a missing table. -- 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]
