[jira] [Commented] (HDFS-11011) Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota'
[ https://issues.apache.org/jira/browse/HDFS-11011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15606119#comment-15606119 ] Hudson commented on HDFS-11011: --- SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10675 (See [https://builds.apache.org/job/Hadoop-trunk-Commit/10675/]) HDFS-11011. Add unit tests for HDFS command 'dfsadmin (xyao: rev 9a8a3864584cec4b545fc974baa110c6839946d4) * (edit) hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestQuota.java * (edit) hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSAdmin.java > Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota' > - > > Key: HDFS-11011 > URL: https://issues.apache.org/jira/browse/HDFS-11011 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: hdfs-client >Reporter: Xiaobing Zhou >Assignee: Xiaobing Zhou > Labels: fs, shell, test > Fix For: 2.8.0, 3.0.0-alpha2 > > Attachments: HDFS-11011.000.patch, HDFS-11011.001.patch, > HDFS-11011.002.patch, HDFS-11011.003.patch, HDFS-11011.004.patch, > HDFS-11011.005.patch, HDFS-11011.006.patch, HDFS-11011.007.patch, > HDFS-11011.008.patch > > > This proposes adding a bunch of unit tests for command 'dfsadmin > setSpaceQuota' and 'dfsadmin clrSpaceQuota'. > 1. test to set space quote using negative number. > 2. test to set and clear space quote, regular usage. > 3. test to set and clear space quote by storage type. > 4. test to set and clear space quote when directory doesn't exist. > 5. test to set and clear space quote when path is a file. > 6. test to set and clear space quote when user has no access right. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-11011) Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota'
[ https://issues.apache.org/jira/browse/HDFS-11011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15606095#comment-15606095 ] Xiaobing Zhou commented on HDFS-11011: -- [~xyao], I filed HDFS-11052 to track the flaky failure. Thanks. > Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota' > - > > Key: HDFS-11011 > URL: https://issues.apache.org/jira/browse/HDFS-11011 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: hdfs-client >Reporter: Xiaobing Zhou >Assignee: Xiaobing Zhou > Labels: fs, shell, test > Fix For: 2.8.0, 3.0.0-alpha2 > > Attachments: HDFS-11011.000.patch, HDFS-11011.001.patch, > HDFS-11011.002.patch, HDFS-11011.003.patch, HDFS-11011.004.patch, > HDFS-11011.005.patch, HDFS-11011.006.patch, HDFS-11011.007.patch, > HDFS-11011.008.patch > > > This proposes adding a bunch of unit tests for command 'dfsadmin > setSpaceQuota' and 'dfsadmin clrSpaceQuota'. > 1. test to set space quote using negative number. > 2. test to set and clear space quote, regular usage. > 3. test to set and clear space quote by storage type. > 4. test to set and clear space quote when directory doesn't exist. > 5. test to set and clear space quote when path is a file. > 6. test to set and clear space quote when user has no access right. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-11011) Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota'
[ https://issues.apache.org/jira/browse/HDFS-11011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15605977#comment-15605977 ] Xiaoyu Yao commented on HDFS-11011: --- Thanks [~xiaobingo] for the update. Patch v008 looks pretty good to me. +1. I will commit it shortly. The unit test failure below is unrelated. Seems like a DNS issue, can you open a new ticket for it as my brief search did not find an open ticket for it? {code} org.apache.hadoop.hdfs.TestPersistBlocks.TestRestartDfsWithFlush End of File Exception between local host is: "48dcc03d04a1/172.17.0.3"; destination host is: "localhost":39481; : java.io.EOFException; For more details see: http://wiki.apache.org/hadoop/EOFException {code} > Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota' > - > > Key: HDFS-11011 > URL: https://issues.apache.org/jira/browse/HDFS-11011 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: hdfs-client >Reporter: Xiaobing Zhou >Assignee: Xiaobing Zhou > Labels: fs, shell, test > Attachments: HDFS-11011.000.patch, HDFS-11011.001.patch, > HDFS-11011.002.patch, HDFS-11011.003.patch, HDFS-11011.004.patch, > HDFS-11011.005.patch, HDFS-11011.006.patch, HDFS-11011.007.patch, > HDFS-11011.008.patch > > > This proposes adding a bunch of unit tests for command 'dfsadmin > setSpaceQuota' and 'dfsadmin clrSpaceQuota'. > 1. test to set space quote using negative number. > 2. test to set and clear space quote, regular usage. > 3. test to set and clear space quote by storage type. > 4. test to set and clear space quote when directory doesn't exist. > 5. test to set and clear space quote when path is a file. > 6. test to set and clear space quote when user has no access right. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-11011) Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota'
[ https://issues.apache.org/jira/browse/HDFS-11011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15604064#comment-15604064 ] Xiaobing Zhou commented on HDFS-11011: -- Thank you [~xyao] for reviews. v008 removed all mini clusters from test functions except {code}testMultipleFilesSmallerThanOneBlock{code} which needs specific conf setting. > Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota' > - > > Key: HDFS-11011 > URL: https://issues.apache.org/jira/browse/HDFS-11011 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: hdfs-client >Reporter: Xiaobing Zhou >Assignee: Xiaobing Zhou > Labels: fs, shell, test > Attachments: HDFS-11011.000.patch, HDFS-11011.001.patch, > HDFS-11011.002.patch, HDFS-11011.003.patch, HDFS-11011.004.patch, > HDFS-11011.005.patch, HDFS-11011.006.patch, HDFS-11011.007.patch, > HDFS-11011.008.patch > > > This proposes adding a bunch of unit tests for command 'dfsadmin > setSpaceQuota' and 'dfsadmin clrSpaceQuota'. > 1. test to set space quote using negative number. > 2. test to set and clear space quote, regular usage. > 3. test to set and clear space quote by storage type. > 4. test to set and clear space quote when directory doesn't exist. > 5. test to set and clear space quote when path is a file. > 6. test to set and clear space quote when user has no access right. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-11011) Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota'
[ https://issues.apache.org/jira/browse/HDFS-11011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15595836#comment-15595836 ] Hadoop QA commented on HDFS-11011: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 26s{color} | {color:blue} Docker mode activated. {color} | | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 2 new or modified test files. {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 8m 56s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 47s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 27s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 55s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 12s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 45s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 42s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 51s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 44s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 44s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 25s{color} | {color:green} hadoop-hdfs-project/hadoop-hdfs: The patch generated 0 new + 38 unchanged - 1 fixed = 38 total (was 39) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 54s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 10s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 51s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 39s{color} | {color:green} the patch passed {color} | | {color:red}-1{color} | {color:red} unit {color} | {color:red} 79m 12s{color} | {color:red} hadoop-hdfs in the patch failed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 18s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black}100m 36s{color} | {color:black} {color} | \\ \\ || Reason || Tests || | Failed junit tests | hadoop.hdfs.TestPersistBlocks | \\ \\ || Subsystem || Report/Notes || | Docker | Image:yetus/hadoop:9560f25 | | JIRA Issue | HDFS-11011 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12834704/HDFS-11011.007.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle | | uname | Linux 48dcc03d04a1 3.13.0-96-generic #143-Ubuntu SMP Mon Aug 29 20:15:20 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh | | git revision | trunk / f63cd78 | | Default Java | 1.8.0_101 | | findbugs | v3.0.0 | | unit | https://builds.apache.org/job/PreCommit-HDFS-Build/17253/artifact/patchprocess/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt | | Test Results | https://builds.apache.org/job/PreCommit-HDFS-Build/17253/testReport/ | | modules | C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs | | Console output | https://builds.apache.org/job/PreCommit-HDFS-Build/17253/console | | Powered by | Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org | This message was automatically generated. > Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota' > - > > Key: HDFS-11011 > URL: https://issues.apache.org/jira/browse/HDFS-11011 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: hdfs-client >
[jira] [Commented] (HDFS-11011) Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota'
[ https://issues.apache.org/jira/browse/HDFS-11011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15595707#comment-15595707 ] Xiaoyu Yao commented on HDFS-11011: --- Thanks [~xiaobingo] and [~liuml07] for working on this. The v007 looks good to me overall. I just have two questions: 1. TestQuota#setUp() is introduced with @Before to start a new cluster instance for each test and assigned to TestQuota#dfsCluster. However, there are a few existing test still using separate local DFSMinicluster instance such as TestQuota#testSpaceCommands, TestQuota#testQuotaByStorageType, etc., can you fix them to use TestQuota#dfsCluster to avoid unnecessary mini cluster creation? 2. Glad to see new tests are added with dedicated test paths. Can you try to run the suite with a single MiniCluster instance that is created with @BeforeClass to save spinup/shutdown cluster for each tests? {code} final Path parent = new Path( PathUtils.getTestPath(getClass()), GenericTestUtils.getMethodName()); {code} > Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota' > - > > Key: HDFS-11011 > URL: https://issues.apache.org/jira/browse/HDFS-11011 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: hdfs-client >Reporter: Xiaobing Zhou >Assignee: Xiaobing Zhou > Labels: fs, shell, test > Attachments: HDFS-11011.000.patch, HDFS-11011.001.patch, > HDFS-11011.002.patch, HDFS-11011.003.patch, HDFS-11011.004.patch, > HDFS-11011.005.patch, HDFS-11011.006.patch, HDFS-11011.007.patch > > > This proposes adding a bunch of unit tests for command 'dfsadmin > setSpaceQuota' and 'dfsadmin clrSpaceQuota'. > 1. test to set space quote using negative number. > 2. test to set and clear space quote, regular usage. > 3. test to set and clear space quote by storage type. > 4. test to set and clear space quote when directory doesn't exist. > 5. test to set and clear space quote when path is a file. > 6. test to set and clear space quote when user has no access right. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-11011) Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota'
[ https://issues.apache.org/jira/browse/HDFS-11011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15595527#comment-15595527 ] Xiaobing Zhou commented on HDFS-11011: -- There were still some check styles missed, v007 fixed them. > Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota' > - > > Key: HDFS-11011 > URL: https://issues.apache.org/jira/browse/HDFS-11011 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: hdfs-client >Reporter: Xiaobing Zhou >Assignee: Xiaobing Zhou > Labels: fs, shell, test > Attachments: HDFS-11011.000.patch, HDFS-11011.001.patch, > HDFS-11011.002.patch, HDFS-11011.003.patch, HDFS-11011.004.patch, > HDFS-11011.005.patch, HDFS-11011.006.patch, HDFS-11011.007.patch > > > This proposes adding a bunch of unit tests for command 'dfsadmin > setSpaceQuota' and 'dfsadmin clrSpaceQuota'. > 1. test to set space quote using negative number. > 2. test to set and clear space quote, regular usage. > 3. test to set and clear space quote by storage type. > 4. test to set and clear space quote when directory doesn't exist. > 5. test to set and clear space quote when path is a file. > 6. test to set and clear space quote when user has no access right. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-11011) Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota'
[ https://issues.apache.org/jira/browse/HDFS-11011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15594212#comment-15594212 ] Hadoop QA commented on HDFS-11011: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 48s{color} | {color:blue} Docker mode activated. {color} | | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 2 new or modified test files. {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 8m 23s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 55s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 28s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 56s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 12s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 56s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 42s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 47s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 42s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 42s{color} | {color:green} the patch passed {color} | | {color:orange}-0{color} | {color:orange} checkstyle {color} | {color:orange} 0m 23s{color} | {color:orange} hadoop-hdfs-project/hadoop-hdfs: The patch generated 6 new + 38 unchanged - 1 fixed = 44 total (was 39) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 50s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 10s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 50s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 37s{color} | {color:green} the patch passed {color} | | {color:red}-1{color} | {color:red} unit {color} | {color:red} 62m 23s{color} | {color:red} hadoop-hdfs in the patch failed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 22s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 83m 42s{color} | {color:black} {color} | \\ \\ || Reason || Tests || | Failed junit tests | hadoop.hdfs.TestMaintenanceState | \\ \\ || Subsystem || Report/Notes || | Docker | Image:yetus/hadoop:9560f25 | | JIRA Issue | HDFS-11011 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12834604/HDFS-11011.006.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle | | uname | Linux 4bfc910f8a4b 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh | | git revision | trunk / 754cb4e | | Default Java | 1.8.0_101 | | findbugs | v3.0.0 | | checkstyle | https://builds.apache.org/job/PreCommit-HDFS-Build/17248/artifact/patchprocess/diff-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt | | unit | https://builds.apache.org/job/PreCommit-HDFS-Build/17248/artifact/patchprocess/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt | | Test Results | https://builds.apache.org/job/PreCommit-HDFS-Build/17248/testReport/ | | modules | C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs | | Console output | https://builds.apache.org/job/PreCommit-HDFS-Build/17248/console | | Powered by | Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org | This message was automatically generated. > Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota' > - > > Key: HDFS-11011 >
[jira] [Commented] (HDFS-11011) Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota'
[ https://issues.apache.org/jira/browse/HDFS-11011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15593785#comment-15593785 ] Mingliang Liu commented on HDFS-11011: -- +1 after the checkstyle warning is addressed. Thanks. > Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota' > - > > Key: HDFS-11011 > URL: https://issues.apache.org/jira/browse/HDFS-11011 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: hdfs-client >Reporter: Xiaobing Zhou >Assignee: Xiaobing Zhou > Labels: fs, shell, test > Attachments: HDFS-11011.000.patch, HDFS-11011.001.patch, > HDFS-11011.002.patch, HDFS-11011.003.patch, HDFS-11011.004.patch, > HDFS-11011.005.patch > > > This proposes adding a bunch of unit tests for command 'dfsadmin > setSpaceQuota' and 'dfsadmin clrSpaceQuota'. > 1. test to set space quote using negative number. > 2. test to set and clear space quote, regular usage. > 3. test to set and clear space quote by storage type. > 4. test to set and clear space quote when directory doesn't exist. > 5. test to set and clear space quote when path is a file. > 6. test to set and clear space quote when user has no access right. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-11011) Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota'
[ https://issues.apache.org/jira/browse/HDFS-11011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15593637#comment-15593637 ] Hadoop QA commented on HDFS-11011: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 11s{color} | {color:blue} Docker mode activated. {color} | | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 2 new or modified test files. {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 8m 19s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 53s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 27s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 0s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 12s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 51s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 40s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 55s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 51s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 51s{color} | {color:green} the patch passed {color} | | {color:orange}-0{color} | {color:orange} checkstyle {color} | {color:orange} 0m 26s{color} | {color:orange} hadoop-hdfs-project/hadoop-hdfs: The patch generated 19 new + 38 unchanged - 1 fixed = 57 total (was 39) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 59s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 10s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 58s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 40s{color} | {color:green} the patch passed {color} | | {color:red}-1{color} | {color:red} unit {color} | {color:red} 67m 0s{color} | {color:red} hadoop-hdfs in the patch failed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 17s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 88m 10s{color} | {color:black} {color} | \\ \\ || Reason || Tests || | Failed junit tests | hadoop.hdfs.server.namenode.web.resources.TestWebHdfsDataLocality | | | hadoop.hdfs.server.namenode.TestDecommissioningStatus | | | hadoop.hdfs.server.namenode.ha.TestDFSUpgradeWithHA | \\ \\ || Subsystem || Report/Notes || | Docker | Image:yetus/hadoop:9560f25 | | JIRA Issue | HDFS-11011 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12834575/HDFS-11011.005.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle | | uname | Linux f28c7974d34c 3.13.0-95-generic #142-Ubuntu SMP Fri Aug 12 17:00:09 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh | | git revision | trunk / 262827c | | Default Java | 1.8.0_101 | | findbugs | v3.0.0 | | checkstyle | https://builds.apache.org/job/PreCommit-HDFS-Build/17243/artifact/patchprocess/diff-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt | | unit | https://builds.apache.org/job/PreCommit-HDFS-Build/17243/artifact/patchprocess/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt | | Test Results | https://builds.apache.org/job/PreCommit-HDFS-Build/17243/testReport/ | | modules | C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs | | Console output | https://builds.apache.org/job/PreCommit-HDFS-Build/17243/console | | Powered by | Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org | This message was automatically generated. > Add unit tests for HDFS command
[jira] [Commented] (HDFS-11011) Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota'
[ https://issues.apache.org/jira/browse/HDFS-11011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15593422#comment-15593422 ] Xiaobing Zhou commented on HDFS-11011: -- Thank you [~liuml07] for the patch. I posted another patch v005 which is based on v002. The reason is that a jumbo reusable function (e.g. v003 and v004) makes code less readable and maintainable. > Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota' > - > > Key: HDFS-11011 > URL: https://issues.apache.org/jira/browse/HDFS-11011 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: hdfs-client >Reporter: Xiaobing Zhou >Assignee: Xiaobing Zhou > Labels: fs, shell, test > Attachments: HDFS-11011.000.patch, HDFS-11011.001.patch, > HDFS-11011.002.patch, HDFS-11011.003.patch, HDFS-11011.004.patch, > HDFS-11011.005.patch > > > This proposes adding a bunch of unit tests for command 'dfsadmin > setSpaceQuota' and 'dfsadmin clrSpaceQuota'. > 1. test to set space quote using negative number. > 2. test to set and clear space quote, regular usage. > 3. test to set and clear space quote by storage type. > 4. test to set and clear space quote when directory doesn't exist. > 5. test to set and clear space quote when path is a file. > 6. test to set and clear space quote when user has no access right. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-11011) Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota'
[ https://issues.apache.org/jira/browse/HDFS-11011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15590323#comment-15590323 ] Hadoop QA commented on HDFS-11011: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 16s{color} | {color:blue} Docker mode activated. {color} | | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 2 new or modified test files. {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 6m 56s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 47s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 27s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 55s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 13s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 40s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 39s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 46s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 40s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 40s{color} | {color:green} the patch passed {color} | | {color:orange}-0{color} | {color:orange} checkstyle {color} | {color:orange} 0m 24s{color} | {color:orange} hadoop-hdfs-project/hadoop-hdfs: The patch generated 9 new + 38 unchanged - 1 fixed = 47 total (was 39) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 47s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 10s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 45s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 36s{color} | {color:green} the patch passed {color} | | {color:red}-1{color} | {color:red} unit {color} | {color:red} 59m 53s{color} | {color:red} hadoop-hdfs in the patch failed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 18s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 78m 23s{color} | {color:black} {color} | \\ \\ || Reason || Tests || | Failed junit tests | hadoop.hdfs.server.namenode.web.resources.TestWebHdfsDataLocality | \\ \\ || Subsystem || Report/Notes || | Docker | Image:yetus/hadoop:9560f25 | | JIRA Issue | HDFS-11011 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12834269/HDFS-11011.004.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle | | uname | Linux 36f17a36a154 3.13.0-92-generic #139-Ubuntu SMP Tue Jun 28 20:42:26 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh | | git revision | trunk / e9c4616 | | Default Java | 1.8.0_101 | | findbugs | v3.0.0 | | checkstyle | https://builds.apache.org/job/PreCommit-HDFS-Build/17226/artifact/patchprocess/diff-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt | | unit | https://builds.apache.org/job/PreCommit-HDFS-Build/17226/artifact/patchprocess/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt | | Test Results | https://builds.apache.org/job/PreCommit-HDFS-Build/17226/testReport/ | | modules | C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs | | Console output | https://builds.apache.org/job/PreCommit-HDFS-Build/17226/console | | Powered by | Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org | This message was automatically generated. > Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota' > - > > Key:
[jira] [Commented] (HDFS-11011) Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota'
[ https://issues.apache.org/jira/browse/HDFS-11011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15586833#comment-15586833 ] Hadoop QA commented on HDFS-11011: -- | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 13s{color} | {color:blue} Docker mode activated. {color} | | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 2 new or modified test files. {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 6m 33s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 45s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 26s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 49s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 12s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 41s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 38s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 44s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 41s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 41s{color} | {color:green} the patch passed {color} | | {color:orange}-0{color} | {color:orange} checkstyle {color} | {color:orange} 0m 24s{color} | {color:orange} hadoop-hdfs-project/hadoop-hdfs: The patch generated 8 new + 38 unchanged - 1 fixed = 46 total (was 39) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 47s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 10s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 45s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 37s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 56m 58s{color} | {color:green} hadoop-hdfs in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 17s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 74m 50s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Image:yetus/hadoop:9560f25 | | JIRA Issue | HDFS-11011 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12834033/HDFS-11011.003.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle | | uname | Linux ca51ef92f664 3.13.0-93-generic #140-Ubuntu SMP Mon Jul 18 21:21:05 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh | | git revision | trunk / 6c348c5 | | Default Java | 1.8.0_101 | | findbugs | v3.0.0 | | checkstyle | https://builds.apache.org/job/PreCommit-HDFS-Build/17209/artifact/patchprocess/diff-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt | | Test Results | https://builds.apache.org/job/PreCommit-HDFS-Build/17209/testReport/ | | modules | C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs | | Console output | https://builds.apache.org/job/PreCommit-HDFS-Build/17209/console | | Powered by | Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org | This message was automatically generated. > Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota' > - > > Key: HDFS-11011 > URL: https://issues.apache.org/jira/browse/HDFS-11011 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: hdfs-client >Reporter: Xiaobing Zhou >Assignee: Xiaobing
[jira] [Commented] (HDFS-11011) Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota'
[ https://issues.apache.org/jira/browse/HDFS-11011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15586641#comment-15586641 ] Xiaobing Zhou commented on HDFS-11011: -- v003 patch is posted, thanks [~liuml07] 1. refactored code 2. added Javadoc 3. fixed check style issues. > Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota' > - > > Key: HDFS-11011 > URL: https://issues.apache.org/jira/browse/HDFS-11011 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: hdfs-client >Reporter: Xiaobing Zhou >Assignee: Xiaobing Zhou > Labels: fs, shell, test > Attachments: HDFS-11011.000.patch, HDFS-11011.001.patch, > HDFS-11011.002.patch, HDFS-11011.003.patch > > > This proposes adding a bunch of unit tests for command 'dfsadmin > setSpaceQuota' and 'dfsadmin clrSpaceQuota'. > 1. test to set space quote using negative number. > 2. test to set and clear space quote, regular usage. > 3. test to set and clear space quote by storage type. > 4. test to set and clear space quote when directory doesn't exist. > 5. test to set and clear space quote when path is a file. > 6. test to set and clear space quote when user has no access right. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-11011) Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota'
[ https://issues.apache.org/jira/browse/HDFS-11011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15583947#comment-15583947 ] Hadoop QA commented on HDFS-11011: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 13s{color} | {color:blue} Docker mode activated. {color} | | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 2 new or modified test files. {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 7m 11s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 46s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 25s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 53s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 12s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 42s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 38s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 47s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 44s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 44s{color} | {color:green} the patch passed {color} | | {color:orange}-0{color} | {color:orange} checkstyle {color} | {color:orange} 0m 23s{color} | {color:orange} hadoop-hdfs-project/hadoop-hdfs: The patch generated 16 new + 38 unchanged - 1 fixed = 54 total (was 39) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 49s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 10s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 47s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 38s{color} | {color:green} the patch passed {color} | | {color:red}-1{color} | {color:red} unit {color} | {color:red} 57m 15s{color} | {color:red} hadoop-hdfs in the patch failed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 19s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 76m 17s{color} | {color:black} {color} | \\ \\ || Reason || Tests || | Failed junit tests | hadoop.hdfs.server.namenode.TestFSImageWithAcl | | | hadoop.hdfs.TestDFSShell | \\ \\ || Subsystem || Report/Notes || | Docker | Image:yetus/hadoop:9560f25 | | JIRA Issue | HDFS-11011 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12833831/HDFS-11011.002.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle | | uname | Linux adb8294ca65d 3.13.0-93-generic #140-Ubuntu SMP Mon Jul 18 21:21:05 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh | | git revision | trunk / f5d9235 | | Default Java | 1.8.0_101 | | findbugs | v3.0.0 | | checkstyle | https://builds.apache.org/job/PreCommit-HDFS-Build/17191/artifact/patchprocess/diff-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt | | unit | https://builds.apache.org/job/PreCommit-HDFS-Build/17191/artifact/patchprocess/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt | | Test Results | https://builds.apache.org/job/PreCommit-HDFS-Build/17191/testReport/ | | modules | C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs | | Console output | https://builds.apache.org/job/PreCommit-HDFS-Build/17191/console | | Powered by | Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org | This message was automatically generated. > Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota' > - > >
[jira] [Commented] (HDFS-11011) Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota'
[ https://issues.apache.org/jira/browse/HDFS-11011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15583864#comment-15583864 ] Mingliang Liu commented on HDFS-11011: -- Thanks for your patch [~xiaobingo]. It looks good to me overall. {code:title=TestDFSAdmin.java} 68 /** 69 * set/clrSpaceQuote are tested in {@link org.apache.hadoop.hdfs.TestQuota}. 70 */ {code} Add one more javadoc for the test class itself? Or this line will be treated as the javadoc for the test class. {code} /** * Test cases for hdfs dfsadmin command. * * Tests for some subcommands are covered elewhere, e.g. set/clrSpaceQuote are tested in {@link org.apache.hadoop.hdfs.TestQuota}. */ {code} {code} 1482/* set space quota */ 1483resetStream(); 1484outs.clear(); 1485ret = whoever.doAs(new PrivilegedExceptionAction() { 1486 @Override 1487 public Integer run() throws Exception { 1488return ToolRunner.run( 1489admin, 1490new String[] {"-setSpaceQuota", "2048", dir.toString()}); 1491 } 1492}); 1493assertEquals(-1, ret); 1494scanIntoList(err, outs); 1495assertThat(outs.get(0), 1496is(allOf(containsString("setSpaceQuota"), 1497containsString("Access denied for user whoever"), 1498containsString("Superuser privilege is required"; 1499 1500/* clear space quota */ 1501resetStream(); 1502outs.clear(); 1503ret = whoever.doAs(new PrivilegedExceptionAction() { 1504 @Override 1505 public Integer run() throws Exception { 1506return ToolRunner.run( 1507admin, 1508new String[] {"-clrSpaceQuota", dir.toString()}); 1509 } 1510}); 1511assertEquals(-1, ret); 1512scanIntoList(err, outs); 1513assertThat(outs.get(0), 1514is(allOf(containsString("clrSpaceQuota"), 1515containsString("Access denied for user whoever"), 1516containsString("Superuser privilege is required"; 1517 } {code} Code duplicates. Can you use a for loop? See https://github.com/apache/hadoop/blob/129125404244f35ee63b8f0491a095371685e9ba/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSAdmin.java#L212-L212 This applies to other places too. By the way, can you update the descriptions with all the newly add test cases? (Simply copy the javadoc for each test case and list them here will work just fine.) > Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota' > - > > Key: HDFS-11011 > URL: https://issues.apache.org/jira/browse/HDFS-11011 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: hdfs-client >Reporter: Xiaobing Zhou >Assignee: Xiaobing Zhou > Labels: fs, shell, test > Attachments: HDFS-11011.000.patch, HDFS-11011.001.patch, > HDFS-11011.002.patch > > > This proposes adding a bunch of unit tests for command 'dfsadmin > setSpaceQuota' and 'dfsadmin clrSpaceQuota'. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-11011) Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota'
[ https://issues.apache.org/jira/browse/HDFS-11011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15583786#comment-15583786 ] Xiaobing Zhou commented on HDFS-11011: -- v002 moved tests to TestQuote. > Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota' > - > > Key: HDFS-11011 > URL: https://issues.apache.org/jira/browse/HDFS-11011 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: hdfs-client >Reporter: Xiaobing Zhou >Assignee: Xiaobing Zhou > Labels: fs, shell, test > Attachments: HDFS-11011.000.patch, HDFS-11011.001.patch, > HDFS-11011.002.patch > > > This proposes adding a bunch of unit tests for command 'dfsadmin > setSpaceQuota' and 'dfsadmin clrSpaceQuota'. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-11011) Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota'
[ https://issues.apache.org/jira/browse/HDFS-11011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15576736#comment-15576736 ] Mingliang Liu commented on HDFS-11011: -- Can we move them together (and add comments in the UT about related places)? > Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota' > - > > Key: HDFS-11011 > URL: https://issues.apache.org/jira/browse/HDFS-11011 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: hdfs-client >Reporter: Xiaobing Zhou >Assignee: Xiaobing Zhou > Labels: fs, shell, test > Attachments: HDFS-11011.000.patch, HDFS-11011.001.patch > > > This proposes adding a bunch of unit tests for command 'dfsadmin > setSpaceQuota' and 'dfsadmin clrSpaceQuota'. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-11011) Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota'
[ https://issues.apache.org/jira/browse/HDFS-11011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15576725#comment-15576725 ] Xiaobing Zhou commented on HDFS-11011: -- Hi Mingliang, they are kind of overlapped, but here it also tested unauthorized access and verified CLI outputs. > Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota' > - > > Key: HDFS-11011 > URL: https://issues.apache.org/jira/browse/HDFS-11011 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: hdfs-client >Reporter: Xiaobing Zhou >Assignee: Xiaobing Zhou > Labels: fs, shell, test > Attachments: HDFS-11011.000.patch, HDFS-11011.001.patch > > > This proposes adding a bunch of unit tests for command 'dfsadmin > setSpaceQuota' and 'dfsadmin clrSpaceQuota'. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-11011) Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota'
[ https://issues.apache.org/jira/browse/HDFS-11011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15576620#comment-15576620 ] Mingliang Liu commented on HDFS-11011: -- Hi Xiaobing, before reviewing the patch, I have a question: {{testQuotaCommands#testQuotaCommands}} is testing the commands; are you adding new tests, or they are duplicate? This seems also true to [HDFS-11016]. > Add unit tests for HDFS command 'dfsadmin -set/clrSpaceQuota' > - > > Key: HDFS-11011 > URL: https://issues.apache.org/jira/browse/HDFS-11011 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: hdfs-client >Reporter: Xiaobing Zhou >Assignee: Xiaobing Zhou > Labels: fs, shell, test > Attachments: HDFS-11011.000.patch, HDFS-11011.001.patch > > > This proposes adding a bunch of unit tests for command 'dfsadmin > setSpaceQuota' and 'dfsadmin clrSpaceQuota'. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org