Octavi00 opened a new pull request, #11426: URL: https://github.com/apache/gravitino/pull/11426
<!-- 1. [Bug report] Trino Gravitino connector throws NullPointerException "Endpoint" when loading metalake with OAuth2 auth [#11365] ### What changes were proposed in this pull request? Added specific exception handling to avoid previous bland nullptr exceptions. ### Why are the changes needed? 1. Allows earlier issue catching (URI syntax tested before http execution) 2. Adds depth to exceptions by specifying common exceptions. Added: UnknownHostException, ConnectException, IOException, NullPointerException, and default Exception to preserve unexpected failures. Fix: #(11365) ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Added two unit tests to prove the specific exceptions' behavior. IMPORTANT: since the bug fix was considered urgent, and I have no previous Kubernetes environment, I chose to utilize unit tests to replicate common exceptions that would allow me to create specific cases for such. - All client tests passed -- 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]
