dennismdejong commented on PR #13670: URL: https://github.com/apache/gravitino/pull/13670#issuecomment-5969834624
Good catches, both accurate. I've updated the PR description: - Qualified the `runJob` HTTP 400 guarantee: it applies when the resource method is reached; with authorization enabled, a null body is rejected with HTTP 403 by the authorization filter before the handler (pre-existing behavior, unchanged by this PR). Removed the "diverging from every other REST operation" claim. - Re-framed the `setCatalog`/`setMetalake` changes as behavior-preserving operation-type reuse — both already reject null bodies before dereferencing the request, so these are not additional null-body fixes. -- 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]
