anujmodi2021 commented on PR #6069: URL: https://github.com/apache/hadoop/pull/6069#issuecomment-1809988857
Regarding the exception handling, I am in agreement to with what @steveloughran has suggested. The whole catch block will work only for the case of AbfsRestOperationException and will blow up if it somehow ends up being not AbfsRestOperationException. It's better to catch the AbfsRestOperationException only for that processing and handle AzureBlobFileSystemEception (which are not AbfsRestOperationException) separately. I have made code changes in this line. Let me know if this looks good @saxenapranav.... Thanks for review -- 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]
