[HDFS-8741] Proper error msg to be printed when invalid operation type is given to WebHDFS operations - ASF JIRA

Details

Type: Improvement
Status: Resolved
Priority: Minor
Resolution: Fixed
Affects Version/s: 3.0.0-alpha2
Fix Version/s: 2.9.0, 3.0.0-alpha4
Component/s: webhdfs
Labels:
None

Target Version/s:

2.9.0, 3.0.0-alpha4
Hadoop Flags:

Reviewed

Description

When wrong operation type is given to WebHDFS operations, following Error message is printed –

For ex: CREATE is called with GET instead of PUT--

HTTP/1.1 400 Bad Request
......
{"RemoteException":{"exception":"IllegalArgumentException","javaClassName":"java.lang.IllegalArgumentException","message":"Invalid value for webhdfs parameter \"op\": No enum constant org.apache.hadoop.hdfs.web.resources.PutOpParam.Op.CREATE"}}

Expected--
Valid Error message to be printed

Options
- Sort By Name
- Sort By Date
- Ascending
- Descending

Attachments

HDFS-8741.001.patch

01/Jan/17 16:24

6 kB

Surendra Singh Lilhore
HDFS-8741.002.patch

20/Feb/17 17:14

6 kB

Surendra Singh Lilhore
HDFS-8741.003.patch

28/Feb/17 08:16

6 kB

Surendra Singh Lilhore
HDFS-8741.004.patch

03/Mar/17 05:17

6 kB

Surendra Singh Lilhore
HDFS-8741.005.patch

03/Mar/17 18:57

6 kB

Surendra Singh Lilhore

Activity

Ascending order - Click to sort in descending order

Hide

Permalink

Surendra Singh Lilhore added a comment - 01/Jan/17 16:24

Thanks Archana T for reporting this issue..
Attached initial patch..
Please review..

Show

Surendra Singh Lilhore added a comment - 01/Jan/17 16:24 Thanks Archana T for reporting this issue.. Attached initial patch.. Please review..

Hide

Permalink

Surendra Singh Lilhore added a comment - 20/Feb/17 17:14

Re-submitting patch to trigger Jenkins build..

Show

Surendra Singh Lilhore added a comment - 20/Feb/17 17:14 Re-submitting patch to trigger Jenkins build..

Hide

Permalink

Hadoop QA added a comment - 20/Feb/17 19:05

-1 overall

Vote	Subsystem	Runtime	Comment
0	reexec	0m 20s	Docker mode activated.
+1	@author	0m 0s	The patch does not contain any @author tags.
+1	test4tests	0m 0s	The patch appears to include 1 new or modified test files.
0	mvndep	0m 28s	Maven dependency ordering for branch
+1	mvninstall	12m 45s	trunk passed
+1	compile	1m 21s	trunk passed
+1	checkstyle	0m 40s	trunk passed
+1	mvnsite	1m 22s	trunk passed
+1	mvneclipse	0m 25s	trunk passed
+1	findbugs	3m 17s	trunk passed
+1	javadoc	1m 3s	trunk passed
0	mvndep	0m 7s	Maven dependency ordering for patch
+1	mvninstall	1m 19s	the patch passed
+1	compile	1m 22s	the patch passed
+1	javac	1m 22s	the patch passed
+1	checkstyle	0m 37s	the patch passed
+1	mvnsite	1m 24s	the patch passed
+1	mvneclipse	0m 21s	the patch passed
+1	whitespace	0m 0s	The patch has no whitespace issues.
+1	findbugs	3m 46s	the patch passed
+1	javadoc	1m 2s	the patch passed
+1	unit	1m 3s	hadoop-hdfs-client in the patch passed.
-1	unit	72m 4s	hadoop-hdfs in the patch failed.
+1	asflicense	0m 19s	The patch does not generate ASF License warnings.
		106m 41s

Reason	Tests
Failed junit tests	hadoop.hdfs.server.datanode.TestDataNodeVolumeFailure
	hadoop.hdfs.server.datanode.TestDataNodeUUID
	hadoop.hdfs.server.datanode.TestDataNodeVolumeFailureReporting
	hadoop.hdfs.server.datanode.TestFsDatasetCache

Subsystem	Report/Notes
Docker	Image:yetus/hadoop:a9ad5d6
JIRA Issue	~~HDFS-8741~~
JIRA Patch URL	https://issues.apache.org/jira/secure/attachment/12853583/HDFS-8741.002.patch
Optional Tests	asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
uname	Linux 11f1eb965621 3.13.0-106-generic #153-Ubuntu SMP Tue Dec 6 15:44:32 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	/testptch/hadoop/patchprocess/precommit/personality/provided.sh
git revision	trunk / 8035749
Default Java	1.8.0_121
findbugs	v3.0.0
unit	https://builds.apache.org/job/PreCommit-HDFS-Build/18407/artifact/patchprocess/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
Test Results	https://builds.apache.org/job/PreCommit-HDFS-Build/18407/testReport/
modules	C: hadoop-hdfs-project/hadoop-hdfs-client hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project
Console output	https://builds.apache.org/job/PreCommit-HDFS-Build/18407/console
Powered by	Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org

This message was automatically generated.

Show

Hadoop QA added a comment - 20/Feb/17 19:05 -1 overall Vote Subsystem Runtime Comment 0 reexec 0m 20s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 1 new or modified test files. 0 mvndep 0m 28s Maven dependency ordering for branch +1 mvninstall 12m 45s trunk passed +1 compile 1m 21s trunk passed +1 checkstyle 0m 40s trunk passed +1 mvnsite 1m 22s trunk passed +1 mvneclipse 0m 25s trunk passed +1 findbugs 3m 17s trunk passed +1 javadoc 1m 3s trunk passed 0 mvndep 0m 7s Maven dependency ordering for patch +1 mvninstall 1m 19s the patch passed +1 compile 1m 22s the patch passed +1 javac 1m 22s the patch passed +1 checkstyle 0m 37s the patch passed +1 mvnsite 1m 24s the patch passed +1 mvneclipse 0m 21s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 3m 46s the patch passed +1 javadoc 1m 2s the patch passed +1 unit 1m 3s hadoop-hdfs-client in the patch passed. -1 unit 72m 4s hadoop-hdfs in the patch failed. +1 asflicense 0m 19s The patch does not generate ASF License warnings. 106m 41s Reason Tests Failed junit tests hadoop.hdfs.server.datanode.TestDataNodeVolumeFailure hadoop.hdfs.server.datanode.TestDataNodeUUID hadoop.hdfs.server.datanode.TestDataNodeVolumeFailureReporting hadoop.hdfs.server.datanode.TestFsDatasetCache Subsystem Report/Notes Docker Image:yetus/hadoop:a9ad5d6 JIRA Issue HDFS-8741 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12853583/HDFS-8741.002.patch Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux 11f1eb965621 3.13.0-106-generic #153-Ubuntu SMP Tue Dec 6 15:44:32 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / 8035749 Default Java 1.8.0_121 findbugs v3.0.0 unit https://builds.apache.org/job/PreCommit-HDFS-Build/18407/artifact/patchprocess/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt Test Results https://builds.apache.org/job/PreCommit-HDFS-Build/18407/testReport/ modules C: hadoop-hdfs-project/hadoop-hdfs-client hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project Console output https://builds.apache.org/job/PreCommit-HDFS-Build/18407/console Powered by Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.

Hide

Permalink

Surendra Singh Lilhore added a comment - 26/Feb/17 04:55

Akira Ajisaka Can you please review..

Show

Surendra Singh Lilhore added a comment - 26/Feb/17 04:55 Akira Ajisaka Can you please review..

Hide

Permalink

Akira Ajisaka added a comment - 28/Feb/17 04:44

Hi Surendra Singh Lilhore, thank you for providing a patch. Mostly looks good to me.

    throw new IllegalArgumentException(str + " not a valid " + Type.GET
          + " operation.");

(minor nit) Would you add "is" between str and not?
I'm +1 if that is addressed.

Show

Akira Ajisaka added a comment - 28/Feb/17 04:44 Hi Surendra Singh Lilhore , thank you for providing a patch. Mostly looks good to me. throw new IllegalArgumentException(str + " not a valid " + Type.GET + " operation." ); (minor nit) Would you add "is" between str and not? I'm +1 if that is addressed.

Hide

Permalink

Surendra Singh Lilhore added a comment - 28/Feb/17 08:18

Thanks Akira Ajisaka for review, attached updated patch...
Please review..

Show

Surendra Singh Lilhore added a comment - 28/Feb/17 08:18 Thanks Akira Ajisaka for review, attached updated patch... Please review..

Hide

Permalink

Yiqun Lin added a comment - 03/Mar/17 03:11 - edited

Thanks Surendra Singh Lilhore for the patch. Some comments for your test.

It would be better to add message into fails(), like following:

+  @Test
+  public void testHttpOpParams() {
+    try {
+      new PostOpParam("TEST");
+      Assert.fail("Construct the PostOpParam with param vaule 'TEST' should be failed.");

Suggest using GenericTestUtils#assertExceptionContains verify the message contained in exception.

Show

Yiqun Lin added a comment - 03/Mar/17 03:11 - edited Thanks Surendra Singh Lilhore for the patch. Some comments for your test. It would be better to add message into fails() , like following: + @Test + public void testHttpOpParams() { + try { + new PostOpParam( "TEST" ); + Assert.fail( "Construct the PostOpParam with param vaule 'TEST' should be failed." ); Suggest using GenericTestUtils#assertExceptionContains verify the message contained in exception.

Hide

Permalink

Surendra Singh Lilhore added a comment - 03/Mar/17 05:19

Thanks Yiqun Lin for review..
attached updated patch, please review..

Show

Surendra Singh Lilhore added a comment - 03/Mar/17 05:19 Thanks Yiqun Lin for review.. attached updated patch, please review..

Hide

Permalink

Yiqun Lin added a comment - 03/Mar/17 06:07

Would you add message into fail() as well? I see it still empty message there.

+  public void testHttpOpParams() {
+    try {
+      new PostOpParam("TEST");
+      Assert.fail();                            <==
+    } catch (IllegalArgumentException e) {
+      GenericTestUtils
+          .assertExceptionContains("TEST is not a valid POST operation.", e);
+    }

Show

Yiqun Lin added a comment - 03/Mar/17 06:07 Would you add message into fail() as well? I see it still empty message there. + public void testHttpOpParams() { + try { + new PostOpParam( "TEST" ); + Assert.fail(); <== + } catch (IllegalArgumentException e) { + GenericTestUtils + .assertExceptionContains( "TEST is not a valid POST operation." , e); + }

Hide

Permalink

Hadoop QA added a comment - 03/Mar/17 07:45

-1 overall

Vote	Subsystem	Runtime	Comment
0	reexec	0m 20s	Docker mode activated.
+1	@author	0m 0s	The patch does not contain any @author tags.
+1	test4tests	0m 0s	The patch appears to include 1 new or modified test files.
0	mvndep	0m 27s	Maven dependency ordering for branch
+1	mvninstall	14m 32s	trunk passed
+1	compile	1m 35s	trunk passed
+1	checkstyle	0m 42s	trunk passed
+1	mvnsite	1m 31s	trunk passed
+1	mvneclipse	0m 28s	trunk passed
+1	findbugs	3m 39s	trunk passed
+1	javadoc	1m 9s	trunk passed
0	mvndep	0m 8s	Maven dependency ordering for patch
+1	mvninstall	1m 30s	the patch passed
+1	compile	1m 31s	the patch passed
+1	javac	1m 31s	the patch passed
+1	checkstyle	0m 39s	the patch passed
+1	mvnsite	1m 32s	the patch passed
+1	mvneclipse	0m 24s	the patch passed
+1	whitespace	0m 0s	The patch has no whitespace issues.
+1	findbugs	3m 46s	the patch passed
+1	javadoc	0m 56s	the patch passed
+1	unit	0m 56s	hadoop-hdfs-client in the patch passed.
-1	unit	109m 48s	hadoop-hdfs in the patch failed.
+1	asflicense	0m 26s	The patch does not generate ASF License warnings.
		147m 33s

Reason	Tests
Failed junit tests	hadoop.hdfs.server.datanode.TestDataNodeVolumeFailure
	hadoop.hdfs.server.datanode.fsdataset.impl.TestSpaceReservation
	hadoop.hdfs.server.datanode.fsdataset.impl.TestLazyPersistReplicaRecovery

Subsystem	Report/Notes
Docker	Image:yetus/hadoop:a9ad5d6
JIRA Issue	~~HDFS-8741~~
JIRA Patch URL	https://issues.apache.org/jira/secure/attachment/12855777/HDFS-8741.004.patch
Optional Tests	asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
uname	Linux e9442bbfc2a8 3.13.0-107-generic #154-Ubuntu SMP Tue Dec 20 09:57:27 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	/testptch/hadoop/patchprocess/precommit/personality/provided.sh
git revision	trunk / 3749152
Default Java	1.8.0_121
findbugs	v3.0.0
unit	https://builds.apache.org/job/PreCommit-HDFS-Build/18533/artifact/patchprocess/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
Test Results	https://builds.apache.org/job/PreCommit-HDFS-Build/18533/testReport/
modules	C: hadoop-hdfs-project/hadoop-hdfs-client hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project
Console output	https://builds.apache.org/job/PreCommit-HDFS-Build/18533/console
Powered by	Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org

This message was automatically generated.

Show

Hadoop QA added a comment - 03/Mar/17 07:45 -1 overall Vote Subsystem Runtime Comment 0 reexec 0m 20s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 1 new or modified test files. 0 mvndep 0m 27s Maven dependency ordering for branch +1 mvninstall 14m 32s trunk passed +1 compile 1m 35s trunk passed +1 checkstyle 0m 42s trunk passed +1 mvnsite 1m 31s trunk passed +1 mvneclipse 0m 28s trunk passed +1 findbugs 3m 39s trunk passed +1 javadoc 1m 9s trunk passed 0 mvndep 0m 8s Maven dependency ordering for patch +1 mvninstall 1m 30s the patch passed +1 compile 1m 31s the patch passed +1 javac 1m 31s the patch passed +1 checkstyle 0m 39s the patch passed +1 mvnsite 1m 32s the patch passed +1 mvneclipse 0m 24s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 3m 46s the patch passed +1 javadoc 0m 56s the patch passed +1 unit 0m 56s hadoop-hdfs-client in the patch passed. -1 unit 109m 48s hadoop-hdfs in the patch failed. +1 asflicense 0m 26s The patch does not generate ASF License warnings. 147m 33s Reason Tests Failed junit tests hadoop.hdfs.server.datanode.TestDataNodeVolumeFailure hadoop.hdfs.server.datanode.fsdataset.impl.TestSpaceReservation hadoop.hdfs.server.datanode.fsdataset.impl.TestLazyPersistReplicaRecovery Subsystem Report/Notes Docker Image:yetus/hadoop:a9ad5d6 JIRA Issue HDFS-8741 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12855777/HDFS-8741.004.patch Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux e9442bbfc2a8 3.13.0-107-generic #154-Ubuntu SMP Tue Dec 20 09:57:27 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / 3749152 Default Java 1.8.0_121 findbugs v3.0.0 unit https://builds.apache.org/job/PreCommit-HDFS-Build/18533/artifact/patchprocess/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt Test Results https://builds.apache.org/job/PreCommit-HDFS-Build/18533/testReport/ modules C: hadoop-hdfs-project/hadoop-hdfs-client hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project Console output https://builds.apache.org/job/PreCommit-HDFS-Build/18533/console Powered by Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.

Hide

Permalink

Surendra Singh Lilhore added a comment - 03/Mar/17 18:57

Sorry I missed first comment..
Attached updated patch. Please review...

Show

Surendra Singh Lilhore added a comment - 03/Mar/17 18:57 Sorry I missed first comment.. Attached updated patch. Please review...

Hide

Permalink

Hadoop QA added a comment - 03/Mar/17 20:48

-1 overall

Vote	Subsystem	Runtime	Comment
0	reexec	0m 13s	Docker mode activated.
+1	@author	0m 0s	The patch does not contain any @author tags.
+1	test4tests	0m 0s	The patch appears to include 1 new or modified test files.
0	mvndep	0m 12s	Maven dependency ordering for branch
+1	mvninstall	14m 58s	trunk passed
+1	compile	1m 46s	trunk passed
+1	checkstyle	0m 47s	trunk passed
+1	mvnsite	1m 40s	trunk passed
+1	mvneclipse	0m 28s	trunk passed
+1	findbugs	3m 36s	trunk passed
+1	javadoc	1m 8s	trunk passed
0	mvndep	0m 7s	Maven dependency ordering for patch
+1	mvninstall	1m 36s	the patch passed
+1	compile	1m 38s	the patch passed
+1	javac	1m 38s	the patch passed
+1	checkstyle	0m 43s	the patch passed
+1	mvnsite	1m 36s	the patch passed
+1	mvneclipse	0m 24s	the patch passed
+1	whitespace	0m 0s	The patch has no whitespace issues.
+1	findbugs	4m 11s	the patch passed
+1	javadoc	1m 1s	the patch passed
+1	unit	1m 3s	hadoop-hdfs-client in the patch passed.
-1	unit	70m 13s	hadoop-hdfs in the patch failed.
+1	asflicense	0m 21s	The patch does not generate ASF License warnings.
		109m 16s

Reason	Tests
Failed junit tests	hadoop.hdfs.TestErasureCodeBenchmarkThroughput

Subsystem	Report/Notes
Docker	Image:yetus/hadoop:a9ad5d6
JIRA Issue	~~HDFS-8741~~
JIRA Patch URL	https://issues.apache.org/jira/secure/attachment/12855944/HDFS-8741.005.patch
Optional Tests	asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
uname	Linux 568a256ecccf 3.13.0-107-generic #154-Ubuntu SMP Tue Dec 20 09:57:27 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	/testptch/hadoop/patchprocess/precommit/personality/provided.sh
git revision	trunk / 490abfb
Default Java	1.8.0_121
findbugs	v3.0.0
unit	https://builds.apache.org/job/PreCommit-HDFS-Build/18545/artifact/patchprocess/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
Test Results	https://builds.apache.org/job/PreCommit-HDFS-Build/18545/testReport/
modules	C: hadoop-hdfs-project/hadoop-hdfs-client hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project
Console output	https://builds.apache.org/job/PreCommit-HDFS-Build/18545/console
Powered by	Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org

This message was automatically generated.

Show

Hadoop QA added a comment - 03/Mar/17 20:48 -1 overall Vote Subsystem Runtime Comment 0 reexec 0m 13s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 1 new or modified test files. 0 mvndep 0m 12s Maven dependency ordering for branch +1 mvninstall 14m 58s trunk passed +1 compile 1m 46s trunk passed +1 checkstyle 0m 47s trunk passed +1 mvnsite 1m 40s trunk passed +1 mvneclipse 0m 28s trunk passed +1 findbugs 3m 36s trunk passed +1 javadoc 1m 8s trunk passed 0 mvndep 0m 7s Maven dependency ordering for patch +1 mvninstall 1m 36s the patch passed +1 compile 1m 38s the patch passed +1 javac 1m 38s the patch passed +1 checkstyle 0m 43s the patch passed +1 mvnsite 1m 36s the patch passed +1 mvneclipse 0m 24s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 4m 11s the patch passed +1 javadoc 1m 1s the patch passed +1 unit 1m 3s hadoop-hdfs-client in the patch passed. -1 unit 70m 13s hadoop-hdfs in the patch failed. +1 asflicense 0m 21s The patch does not generate ASF License warnings. 109m 16s Reason Tests Failed junit tests hadoop.hdfs.TestErasureCodeBenchmarkThroughput Subsystem Report/Notes Docker Image:yetus/hadoop:a9ad5d6 JIRA Issue HDFS-8741 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12855944/HDFS-8741.005.patch Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux 568a256ecccf 3.13.0-107-generic #154-Ubuntu SMP Tue Dec 20 09:57:27 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / 490abfb Default Java 1.8.0_121 findbugs v3.0.0 unit https://builds.apache.org/job/PreCommit-HDFS-Build/18545/artifact/patchprocess/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt Test Results https://builds.apache.org/job/PreCommit-HDFS-Build/18545/testReport/ modules C: hadoop-hdfs-project/hadoop-hdfs-client hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project Console output https://builds.apache.org/job/PreCommit-HDFS-Build/18545/console Powered by Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.

Hide

Permalink

Yiqun Lin added a comment - 06/Mar/17 02:08

The test failure is not related. LGTM, +1.
Hi Akira Ajisaka, do you have further comment for the latest patch, if not, I will commit shortly. Thanks.

Show

Yiqun Lin added a comment - 06/Mar/17 02:08 The test failure is not related. LGTM, +1. Hi Akira Ajisaka , do you have further comment for the latest patch, if not, I will commit shortly. Thanks.

Hide

Permalink

Akira Ajisaka added a comment - 06/Mar/17 04:39

+1, thanks Yiqun Lin and Surendra Singh Lilhore.

Show

Akira Ajisaka added a comment - 06/Mar/17 04:39 +1, thanks Yiqun Lin and Surendra Singh Lilhore .

Hide

Permalink

Yiqun Lin added a comment - 06/Mar/17 11:08 - edited

Committed this to trunk and branch-2. Thanks Surendra Singh Lilhore for the contribution and thanks Akira Ajisaka for the review!

Show

Yiqun Lin added a comment - 06/Mar/17 11:08 - edited Committed this to trunk and branch-2. Thanks Surendra Singh Lilhore for the contribution and thanks Akira Ajisaka for the review!

Hide

Permalink

Hudson added a comment - 06/Mar/17 11:32

SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #11352 (See https://builds.apache.org/job/Hadoop-trunk-Commit/11352/)
~~HDFS-8741~~. Proper error msg to be printed when invalid operation type is (yqlin: rev 3536ce031ca780d6de83cf67779f571a0142ccc8)

(edit) hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/resources/DeleteOpParam.java
(edit) hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/resources/PostOpParam.java
(edit) hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/resources/GetOpParam.java
(edit) hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/resources/PutOpParam.java
(edit) hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/web/resources/TestParam.java

Show

Hudson added a comment - 06/Mar/17 11:32 SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #11352 (See https://builds.apache.org/job/Hadoop-trunk-Commit/11352/ ) HDFS-8741 . Proper error msg to be printed when invalid operation type is (yqlin: rev 3536ce031ca780d6de83cf67779f571a0142ccc8) (edit) hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/resources/DeleteOpParam.java (edit) hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/resources/PostOpParam.java (edit) hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/resources/GetOpParam.java (edit) hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/resources/PutOpParam.java (edit) hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/web/resources/TestParam.java

Hide

Permalink

Surendra Singh Lilhore added a comment - 06/Mar/17 12:36

Thanks Yiqun Lin for review and commit. Thanks Akira Ajisaka for review..

Show

Surendra Singh Lilhore added a comment - 06/Mar/17 12:36 Thanks Yiqun Lin for review and commit. Thanks Akira Ajisaka for review..

People

Assignee:

Surendra Singh Lilhore

Reporter:

Archana T

Votes:

0 Vote for this issue

Watchers:

7 Start watching this issue

Dates

Created:

09/Jul/15 03:58

Updated:

06/Mar/17 12:36

Resolved:

06/Mar/17 11:07

Development

Agile

View on Board