anujmodi2021 commented on code in PR #6879:
URL: https://github.com/apache/hadoop/pull/6879#discussion_r1696681697
##########
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/AzureBlobFileSystem.java:
##########
@@ -1400,6 +1418,29 @@ private FileStatus tryGetFileStatus(final Path f,
TracingContext tracingContext)
}
}
+ private boolean tryGetIsNamespaceEnabled(TracingContext tracingContext)
+ throws AzureBlobFileSystemException{
+ try {
+ return getIsNamespaceEnabled(tracingContext);
+ } catch (AbfsRestOperationException ex) {
+ /*
+ * Exception will be thrown for any non 400 error code.
+ * If status code is in 4xx range, it means it's an HNS account.
+ * If status code is in 5xx range, it means nothing can be inferred.
+ * In case of network errors status code will be -1.
+ */
+ int statusCode = ex.getStatusCode();
+ if (statusCode > HTTP_BAD_REQUEST && statusCode < HTTP_INTERNAL_ERROR) {
+ LOG.debug("getNamespace failed with non 400 user error", ex);
+ statIncrement(ERROR_IGNORED);
+ return true;
+ }
+ throw ex;
+ } catch (AzureBlobFileSystemException ex) {
Review Comment:
Makes sense, removed the catch block
--
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: common-issues-unsubscr...@hadoop.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org
