nevzheng commented on PR #11959: URL: https://github.com/apache/gravitino/pull/11959#issuecomment-5005999528
@roryqi @yuqi1129 Thanks for the feedback. I’ve addressed and resolved the review threads and updated the PR description to clarify the scope: - The primary fix is the `NoSuchWarehouseException` case from #11943. - Invalid REST warehouse selectors now fail at creation with an actionable HTTP 400. - Transport/503 handling is limited to REST initialization so a temporary dependency outage is not confused with invalid configuration. - Broader Iceberg REST response mapping is explicitly left as future work. The targeted unit and integration coverage is included. Could you please take a final look and formally approve if the behavior and scope look good? CI is still running, with no failures currently. -- 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]
