anmolanmol1234 commented on code in PR #7765:
URL: https://github.com/apache/hadoop/pull/7765#discussion_r2195592571
##########
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/AzureBlobFileSystemStore.java:
##########
@@ -392,30 +392,40 @@ public boolean getIsNamespaceEnabled(TracingContext
tracingContext)
return getNamespaceEnabledInformationFromServer(tracingContext);
}
+ /**
+ * In case the namespace configuration is not set or invalid, this method
will
+ * make a call to the server to determine if namespace is enabled or not.
+ * This method is synchronized to ensure that only one thread
+ * is making the call to the server to determine the namespace
+ *
+ * @param tracingContext tracing context
+ * @return true if namespace is enabled, false otherwise.
+ * @throws AzureBlobFileSystemException server errors.
+ */
private synchronized boolean getNamespaceEnabledInformationFromServer(
final TracingContext tracingContext) throws AzureBlobFileSystemException
{
- if (abfsConfiguration.getIsNamespaceEnabledAccount() != Trilean.UNKNOWN) {
+ if (getAbfsConfiguration().getIsNamespaceEnabledAccount() !=
Trilean.UNKNOWN) {
return isNamespaceEnabled();
}
try {
LOG.debug("Get root ACL status");
getClient(AbfsServiceType.DFS).getAclStatus(AbfsHttpConstants.ROOT_PATH,
tracingContext);
// If getAcl succeeds, namespace is enabled.
- setNamespaceEnabled(Trilean.TRUE);
+ setNamespaceEnabled(true);
Review Comment:
This is optional as you mentioned.
--
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
