gaborkaszab commented on PR #14035: URL: https://github.com/apache/iceberg/pull/14035#issuecomment-3421164367
The changes I made recently on this PR: - Rebased with main to have the Route class refactor - Noticed, the current test mocks the adapter in RESTSessionCatalog and as a result the query doesn't go through HTTPClient and RESTCatalogServlet. Made some adjustments on the tests to be able to mock the adapter within the servlet instead of the one within RESTSessionCatalog to not test the whole chain. @amogh-jahagirdar Do you think you have some capacity to take a look? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
