[PR] fix(rest-server): use Java's uppercase resourceType in error responses [paimon-rust]

["jackylee-ch (via GitHub)" <gi...@apache.org>]

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

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

Re: [PR] fix(rest-server): use Java's uppercase resourceType in error responses [paimon-rust]

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

JingsongLi merged PR #830:
URL: https://github.com/apache/paimon-rust/pull/830


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