virajith commented on a change in pull request #1711: Hadoop 16455
URL: https://github.com/apache/hadoop/pull/1711#discussion_r346112996
##########
File path:
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/AzureBlobFileSystem.java
##########
@@ -862,8 +862,13 @@ public AclStatus getAclStatus(final Path path) throws
IOException {
*/
@Override
public void access(final Path path, FsAction mode) throws IOException {
Review comment:
Should {{getIsNamespaceEnabled()}} be checked? In the test, you use
modifyAclEntries which requires hierarchical namespace to be enabled. Shouldn't
this be the case for access() as well?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
