yuqi1129 commented on PR #12880: URL: https://github.com/apache/gravitino/pull/12880#issuecomment-5523832602
Addressed this systemically through `BaseExceptionHandler`, so resource families without a specialized `UnsupportedOperationException` branch now return 501 instead of falling through to 500. I also audited the exception hierarchy and kept more specific semantics ahead of that fallback: invalid statistics targets return 400, and `UnmodifiableStatisticException` returns 409. The six statistics operations are now covered together in one regression test. The authorization-disabled filter retains its existing 405 contract through a dedicated helper, as called out in the original report. Web v2 now detects the stable application error code as well as 501, preserving compatibility with older servers while not swallowing a real framework-generated 405. -- 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]
