Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id E5EA4200CDF for ; Thu, 17 Aug 2017 20:28:59 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id E45A016B75F; Thu, 17 Aug 2017 18:28:59 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id B2B1816B75B for ; Thu, 17 Aug 2017 20:28:58 +0200 (CEST) Received: (qmail 98073 invoked by uid 500); 17 Aug 2017 18:28:55 -0000 Mailing-List: contact common-commits-help@hadoop.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Delivered-To: mailing list common-commits@hadoop.apache.org Received: (qmail 96685 invoked by uid 99); 17 Aug 2017 18:28:54 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 17 Aug 2017 18:28:54 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 7D725F565A; Thu, 17 Aug 2017 18:28:53 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: wangda@apache.org To: common-commits@hadoop.apache.org Date: Thu, 17 Aug 2017 18:29:01 -0000 Message-Id: <3f2880ff7b2d499ebafde3e98ce3ae4b@git.apache.org> In-Reply-To: <1cd4c258bf744fc4a87eacb5daab2c0c@git.apache.org> References: <1cd4c258bf744fc4a87eacb5daab2c0c@git.apache.org> X-Mailer: ASF-Git Admin Mailer Subject: [09/37] hadoop git commit: HDFS-12250. Reduce usage of FsPermissionExtension in unit tests. Contributed by Chris Douglas. archived-at: Thu, 17 Aug 2017 18:29:00 -0000 HDFS-12250. Reduce usage of FsPermissionExtension in unit tests. Contributed by Chris Douglas. Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/dd7916d3 Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/dd7916d3 Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/dd7916d3 Branch: refs/heads/YARN-3926 Commit: dd7916d3cd5d880d0b257d229f43f10feff04c93 Parents: f9a0e23 Author: Andrew Wang Authored: Thu Aug 17 09:35:36 2017 -0700 Committer: Andrew Wang Committed: Thu Aug 17 09:35:36 2017 -0700 ---------------------------------------------------------------------- .../hadoop/fs/permission/FsPermission.java | 2 +- .../org/apache/hadoop/fs/shell/AclCommands.java | 6 ++--- .../hadoop/fs/shell/CommandWithDestination.java | 4 ++-- .../java/org/apache/hadoop/fs/shell/Ls.java | 4 ++-- .../fs/http/client/BaseTestHttpFSWith.java | 1 + .../org/apache/hadoop/hdfs/TestDFSShell.java | 24 ++++++++++---------- .../hdfs/server/namenode/FSAclBaseTest.java | 6 +++++ .../ClientDistributedCacheManager.java | 6 ++--- .../apache/hadoop/fs/adl/TestGetFileStatus.java | 1 + .../hadoop/tools/CopyListingFileStatus.java | 4 ++-- .../apache/hadoop/tools/util/DistCpUtils.java | 4 +--- 11 files changed, 33 insertions(+), 29 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hadoop/blob/dd7916d3/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/FsPermission.java ---------------------------------------------------------------------- diff --git a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/FsPermission.java b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/FsPermission.java index 23692de..031092b 100644 --- a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/FsPermission.java +++ b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/FsPermission.java @@ -163,7 +163,7 @@ public class FsPermission implements Writable, Serializable, */ public static FsPermission read(DataInput in) throws IOException { FsPermission p = new FsPermission(); - p.readFields(in); + p.fromShort(in.readShort()); return p; } http://git-wip-us.apache.org/repos/asf/hadoop/blob/dd7916d3/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/shell/AclCommands.java ---------------------------------------------------------------------- diff --git a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/shell/AclCommands.java b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/shell/AclCommands.java index a5e386c..701c9de 100644 --- a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/shell/AclCommands.java +++ b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/shell/AclCommands.java @@ -86,9 +86,9 @@ class AclCommands extends FsCommand { (perm.getOtherAction().implies(FsAction.EXECUTE) ? "t" : "T")); } - AclStatus aclStatus = null; - List entries = null; - if (perm.getAclBit()) { + final AclStatus aclStatus; + final List entries; + if (item.stat.hasAcl()) { aclStatus = item.fs.getAclStatus(item.path); entries = aclStatus.getEntries(); } else { http://git-wip-us.apache.org/repos/asf/hadoop/blob/dd7916d3/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/shell/CommandWithDestination.java ---------------------------------------------------------------------- diff --git a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/shell/CommandWithDestination.java b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/shell/CommandWithDestination.java index 2a483c0..0bd4882 100644 --- a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/shell/CommandWithDestination.java +++ b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/shell/CommandWithDestination.java @@ -444,8 +444,8 @@ abstract class CommandWithDestination extends FsCommand { src.stat.getPermission()); } if (shouldPreserve(FileAttribute.ACL)) { - FsPermission perm = src.stat.getPermission(); - if (perm.getAclBit()) { + if (src.stat.hasAcl()) { + FsPermission perm = src.stat.getPermission(); List srcEntries = src.fs.getAclStatus(src.path).getEntries(); List srcFullEntries = http://git-wip-us.apache.org/repos/asf/hadoop/blob/dd7916d3/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/shell/Ls.java ---------------------------------------------------------------------- diff --git a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/shell/Ls.java b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/shell/Ls.java index 221b3cb..a2d5017 100644 --- a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/shell/Ls.java +++ b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/shell/Ls.java @@ -255,7 +255,7 @@ class Ls extends FsCommand { ContentSummary contentSummary = item.fs.getContentSummary(item.path); String line = String.format(lineFormat, (stat.isDirectory() ? "d" : "-"), - stat.getPermission() + (stat.getPermission().getAclBit() ? "+" : " "), + stat.getPermission() + (stat.hasAcl() ? "+" : " "), (stat.isFile() ? stat.getReplication() : "-"), stat.getOwner(), stat.getGroup(), @@ -269,7 +269,7 @@ class Ls extends FsCommand { } else { String line = String.format(lineFormat, (stat.isDirectory() ? "d" : "-"), - stat.getPermission() + (stat.getPermission().getAclBit() ? "+" : " "), + stat.getPermission() + (stat.hasAcl() ? "+" : " "), (stat.isFile() ? stat.getReplication() : "-"), stat.getOwner(), stat.getGroup(), http://git-wip-us.apache.org/repos/asf/hadoop/blob/dd7916d3/hadoop-hdfs-project/hadoop-hdfs-httpfs/src/test/java/org/apache/hadoop/fs/http/client/BaseTestHttpFSWith.java ---------------------------------------------------------------------- diff --git a/hadoop-hdfs-project/hadoop-hdfs-httpfs/src/test/java/org/apache/hadoop/fs/http/client/BaseTestHttpFSWith.java b/hadoop-hdfs-project/hadoop-hdfs-httpfs/src/test/java/org/apache/hadoop/fs/http/client/BaseTestHttpFSWith.java index 553bbce..2cd8934 100644 --- a/hadoop-hdfs-project/hadoop-hdfs-httpfs/src/test/java/org/apache/hadoop/fs/http/client/BaseTestHttpFSWith.java +++ b/hadoop-hdfs-project/hadoop-hdfs-httpfs/src/test/java/org/apache/hadoop/fs/http/client/BaseTestHttpFSWith.java @@ -859,6 +859,7 @@ public abstract class BaseTestHttpFSWith extends HFSTestCase { FileStatus expectedFileStatus = expected.getFileStatus(path); FileStatus actualFileStatus = actual.getFileStatus(path); assertEquals(actualFileStatus.hasAcl(), expectedFileStatus.hasAcl()); + // backwards compat assertEquals(actualFileStatus.getPermission().getAclBit(), expectedFileStatus.getPermission().getAclBit()); } http://git-wip-us.apache.org/repos/asf/hadoop/blob/dd7916d3/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDFSShell.java ---------------------------------------------------------------------- diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDFSShell.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDFSShell.java index 27d41b4..9ae49aa 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDFSShell.java +++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDFSShell.java @@ -2189,7 +2189,7 @@ public class TestDFSShell { assertTrue(xattrs.isEmpty()); List acls = dfs.getAclStatus(target1).getEntries(); assertTrue(acls.isEmpty()); - assertFalse(targetPerm.getAclBit()); + assertFalse(targetStatus.hasAcl()); // -ptop Path target2 = new Path(hdfsTestDir, "targetfile2"); @@ -2208,7 +2208,7 @@ public class TestDFSShell { assertTrue(xattrs.isEmpty()); acls = dfs.getAclStatus(target2).getEntries(); assertTrue(acls.isEmpty()); - assertFalse(targetPerm.getAclBit()); + assertFalse(targetStatus.hasAcl()); // -ptopx Path target3 = new Path(hdfsTestDir, "targetfile3"); @@ -2229,7 +2229,7 @@ public class TestDFSShell { assertArrayEquals(TRUSTED_A1_VALUE, xattrs.get(TRUSTED_A1)); acls = dfs.getAclStatus(target3).getEntries(); assertTrue(acls.isEmpty()); - assertFalse(targetPerm.getAclBit()); + assertFalse(targetStatus.hasAcl()); // -ptopa Path target4 = new Path(hdfsTestDir, "targetfile4"); @@ -2248,7 +2248,7 @@ public class TestDFSShell { assertTrue(xattrs.isEmpty()); acls = dfs.getAclStatus(target4).getEntries(); assertFalse(acls.isEmpty()); - assertTrue(targetPerm.getAclBit()); + assertTrue(targetStatus.hasAcl()); assertEquals(dfs.getAclStatus(src), dfs.getAclStatus(target4)); // -ptoa (verify -pa option will preserve permissions also) @@ -2268,7 +2268,7 @@ public class TestDFSShell { assertTrue(xattrs.isEmpty()); acls = dfs.getAclStatus(target5).getEntries(); assertFalse(acls.isEmpty()); - assertTrue(targetPerm.getAclBit()); + assertTrue(targetStatus.hasAcl()); assertEquals(dfs.getAclStatus(src), dfs.getAclStatus(target5)); } finally { if (null != shell) { @@ -2480,7 +2480,7 @@ public class TestDFSShell { assertTrue(xattrs.isEmpty()); List acls = dfs.getAclStatus(targetDir1).getEntries(); assertTrue(acls.isEmpty()); - assertFalse(targetPerm.getAclBit()); + assertFalse(targetStatus.hasAcl()); // -ptop Path targetDir2 = new Path(hdfsTestDir, "targetDir2"); @@ -2499,7 +2499,7 @@ public class TestDFSShell { assertTrue(xattrs.isEmpty()); acls = dfs.getAclStatus(targetDir2).getEntries(); assertTrue(acls.isEmpty()); - assertFalse(targetPerm.getAclBit()); + assertFalse(targetStatus.hasAcl()); // -ptopx Path targetDir3 = new Path(hdfsTestDir, "targetDir3"); @@ -2520,7 +2520,7 @@ public class TestDFSShell { assertArrayEquals(TRUSTED_A1_VALUE, xattrs.get(TRUSTED_A1)); acls = dfs.getAclStatus(targetDir3).getEntries(); assertTrue(acls.isEmpty()); - assertFalse(targetPerm.getAclBit()); + assertFalse(targetStatus.hasAcl()); // -ptopa Path targetDir4 = new Path(hdfsTestDir, "targetDir4"); @@ -2539,7 +2539,7 @@ public class TestDFSShell { assertTrue(xattrs.isEmpty()); acls = dfs.getAclStatus(targetDir4).getEntries(); assertFalse(acls.isEmpty()); - assertTrue(targetPerm.getAclBit()); + assertTrue(targetStatus.hasAcl()); assertEquals(dfs.getAclStatus(srcDir), dfs.getAclStatus(targetDir4)); // -ptoa (verify -pa option will preserve permissions also) @@ -2559,7 +2559,7 @@ public class TestDFSShell { assertTrue(xattrs.isEmpty()); acls = dfs.getAclStatus(targetDir5).getEntries(); assertFalse(acls.isEmpty()); - assertTrue(targetPerm.getAclBit()); + assertTrue(targetStatus.hasAcl()); assertEquals(dfs.getAclStatus(srcDir), dfs.getAclStatus(targetDir5)); } finally { if (shell != null) { @@ -2615,7 +2615,7 @@ public class TestDFSShell { assertTrue(perm.equals(targetPerm)); List acls = dfs.getAclStatus(target1).getEntries(); assertTrue(acls.isEmpty()); - assertFalse(targetPerm.getAclBit()); + assertFalse(targetStatus.hasAcl()); // -ptopa preserves both sticky bit and ACL Path target2 = new Path(hdfsTestDir, "targetfile2"); @@ -2632,7 +2632,7 @@ public class TestDFSShell { assertTrue(perm.equals(targetPerm)); acls = dfs.getAclStatus(target2).getEntries(); assertFalse(acls.isEmpty()); - assertTrue(targetPerm.getAclBit()); + assertTrue(targetStatus.hasAcl()); assertEquals(dfs.getAclStatus(src), dfs.getAclStatus(target2)); } finally { if (null != shell) { http://git-wip-us.apache.org/repos/asf/hadoop/blob/dd7916d3/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/FSAclBaseTest.java ---------------------------------------------------------------------- diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/FSAclBaseTest.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/FSAclBaseTest.java index 93a83fd..ee92217 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/FSAclBaseTest.java +++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/FSAclBaseTest.java @@ -32,6 +32,7 @@ import java.util.Map; import org.apache.hadoop.conf.Configuration; import org.apache.hadoop.fs.CommonConfigurationKeys; +import org.apache.hadoop.fs.FileStatus; import org.apache.hadoop.fs.FileSystem; import org.apache.hadoop.fs.Path; import org.apache.hadoop.fs.permission.AclEntry; @@ -886,6 +887,11 @@ public abstract class FSAclBaseTest { FsPermission perm = inode.getFsPermission(); assertNotNull(perm); assertEquals(0755, perm.toShort()); + FileStatus stat = fs.getFileStatus(path); + assertFalse(stat.hasAcl()); + assertFalse(stat.isEncrypted()); + assertFalse(stat.isErasureCoded()); + // backwards-compat check assertEquals(0755, perm.toExtendedShort()); assertAclFeature(false); } http://git-wip-us.apache.org/repos/asf/hadoop/blob/dd7916d3/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/filecache/ClientDistributedCacheManager.java ---------------------------------------------------------------------- diff --git a/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/filecache/ClientDistributedCacheManager.java b/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/filecache/ClientDistributedCacheManager.java index 9f8edb5..ada14db 100644 --- a/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/filecache/ClientDistributedCacheManager.java +++ b/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/filecache/ClientDistributedCacheManager.java @@ -28,7 +28,6 @@ import org.apache.hadoop.fs.FileStatus; import org.apache.hadoop.fs.FileSystem; import org.apache.hadoop.fs.Path; import org.apache.hadoop.fs.permission.FsAction; -import org.apache.hadoop.fs.permission.FsPermission; import org.apache.hadoop.mapreduce.MRJobConfig; import org.apache.hadoop.mapreduce.security.TokenCache; import org.apache.hadoop.security.Credentials; @@ -293,7 +292,6 @@ public class ClientDistributedCacheManager { private static boolean checkPermissionOfOther(FileSystem fs, Path path, FsAction action, Map statCache) throws IOException { FileStatus status = getFileStatus(fs, path.toUri(), statCache); - FsPermission perms = status.getPermission(); // Encrypted files are always treated as private. This stance has two // important side effects. The first is that the encrypted files will be @@ -302,8 +300,8 @@ public class ClientDistributedCacheManager { // world readable permissions that is stored in an encryption zone from // being localized as a publicly shared file with world readable // permissions. - if (!perms.getEncryptedBit()) { - FsAction otherAction = perms.getOtherAction(); + if (!status.isEncrypted()) { + FsAction otherAction = status.getPermission().getOtherAction(); if (otherAction.implies(action)) { return true; } http://git-wip-us.apache.org/repos/asf/hadoop/blob/dd7916d3/hadoop-tools/hadoop-azure-datalake/src/test/java/org/apache/hadoop/fs/adl/TestGetFileStatus.java ---------------------------------------------------------------------- diff --git a/hadoop-tools/hadoop-azure-datalake/src/test/java/org/apache/hadoop/fs/adl/TestGetFileStatus.java b/hadoop-tools/hadoop-azure-datalake/src/test/java/org/apache/hadoop/fs/adl/TestGetFileStatus.java index d9e22db..95c2363 100644 --- a/hadoop-tools/hadoop-azure-datalake/src/test/java/org/apache/hadoop/fs/adl/TestGetFileStatus.java +++ b/hadoop-tools/hadoop-azure-datalake/src/test/java/org/apache/hadoop/fs/adl/TestGetFileStatus.java @@ -98,4 +98,5 @@ public class TestGetFileStatus extends AdlMockWebServer { Assert.assertFalse(fileStatus.hasAcl()); Assert.assertFalse(fileStatus.getPermission().getAclBit()); } + } http://git-wip-us.apache.org/repos/asf/hadoop/blob/dd7916d3/hadoop-tools/hadoop-distcp/src/main/java/org/apache/hadoop/tools/CopyListingFileStatus.java ---------------------------------------------------------------------- diff --git a/hadoop-tools/hadoop-distcp/src/main/java/org/apache/hadoop/tools/CopyListingFileStatus.java b/hadoop-tools/hadoop-distcp/src/main/java/org/apache/hadoop/tools/CopyListingFileStatus.java index 29c59ac..138b491 100644 --- a/hadoop-tools/hadoop-distcp/src/main/java/org/apache/hadoop/tools/CopyListingFileStatus.java +++ b/hadoop-tools/hadoop-distcp/src/main/java/org/apache/hadoop/tools/CopyListingFileStatus.java @@ -280,7 +280,7 @@ public final class CopyListingFileStatus implements Writable { out.writeLong(getBlockSize()); out.writeLong(getModificationTime()); out.writeLong(getAccessTime()); - getPermission().write(out); + out.writeShort(getPermission().toShort()); Text.writeString(out, getOwner(), Text.DEFAULT_MAX_LEN); Text.writeString(out, getGroup(), Text.DEFAULT_MAX_LEN); if (aclEntries != null) { @@ -330,7 +330,7 @@ public final class CopyListingFileStatus implements Writable { blocksize = in.readLong(); modificationTime = in.readLong(); accessTime = in.readLong(); - permission.readFields(in); + permission.fromShort(in.readShort()); owner = Text.readString(in, Text.DEFAULT_MAX_LEN); group = Text.readString(in, Text.DEFAULT_MAX_LEN); byte aclEntriesSize = in.readByte(); http://git-wip-us.apache.org/repos/asf/hadoop/blob/dd7916d3/hadoop-tools/hadoop-distcp/src/main/java/org/apache/hadoop/tools/util/DistCpUtils.java ---------------------------------------------------------------------- diff --git a/hadoop-tools/hadoop-distcp/src/main/java/org/apache/hadoop/tools/util/DistCpUtils.java b/hadoop-tools/hadoop-distcp/src/main/java/org/apache/hadoop/tools/util/DistCpUtils.java index dbe750a..2b3b529 100644 --- a/hadoop-tools/hadoop-distcp/src/main/java/org/apache/hadoop/tools/util/DistCpUtils.java +++ b/hadoop-tools/hadoop-distcp/src/main/java/org/apache/hadoop/tools/util/DistCpUtils.java @@ -31,7 +31,6 @@ import org.apache.hadoop.fs.Path; import org.apache.hadoop.fs.XAttr; import org.apache.hadoop.fs.permission.AclEntry; import org.apache.hadoop.fs.permission.AclUtil; -import org.apache.hadoop.fs.permission.FsPermission; import org.apache.hadoop.hdfs.DistributedFileSystem; import org.apache.hadoop.io.SequenceFile; import org.apache.hadoop.io.Text; @@ -403,8 +402,7 @@ public class DistCpUtils { CopyListingFileStatus copyListingFileStatus = new CopyListingFileStatus(fileStatus, chunkOffset, chunkLength); if (preserveAcls) { - FsPermission perm = fileStatus.getPermission(); - if (perm.getAclBit()) { + if (fileStatus.hasAcl()) { List aclEntries = fileSystem.getAclStatus( fileStatus.getPath()).getEntries(); copyListingFileStatus.setAclEntries(aclEntries); --------------------------------------------------------------------- To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org For additional commands, e-mail: common-commits-help@hadoop.apache.org