[
https://issues.apache.org/jira/browse/HDFS-5939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13909084#comment-13909084
]
Yongjun Zhang commented on HDFS-5939:
-------------------------------------
Hi Haohui,
Thanks for your review earlier.
I gave some more thought and decide that it's a good idea to remove the message
printing, though I still think there is value to have the message. The reasons
are below.
1. If we have this kind of delimiter message for all unit testcase, it would be
nice. Adding it for one unit testcase makes it inconsistent across the board.
2. If we really want this kind of delimiter message across the board, from
engineering point of view, it would be nice to have it automated, rather than
have to have developer to add it for each single one.
For example, Junit framwork could help to add this kind of message when
running the test.
I will post a new version a bit later today.
Thanks again for the review, I learned quite a bit from it.
> WebHdfs returns misleading error code and logs nothing if trying to create a
> file with no DNs in cluster
> --------------------------------------------------------------------------------------------------------
>
> Key: HDFS-5939
> URL: https://issues.apache.org/jira/browse/HDFS-5939
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: hdfs-client
> Affects Versions: 2.3.0
> Reporter: Yongjun Zhang
> Assignee: Yongjun Zhang
> Attachments: HDFS-5939.001.patch, HDFS-5939.002.patch,
> HDFS-5939.003.patch, HDFS-5939.004.patch, HDFS-5939.005.patch
>
>
> When trying to access hdfs via webhdfs, and when datanode is dead, user will
> see an exception below without any clue that it's caused by dead datanode:
> $ curl -i -X PUT
> ".../webhdfs/v1/t1?op=CREATE&user.name=<userName>&overwrite=false"
> ...
> {"RemoteException":{"exception":"IllegalArgumentException","javaClassName":"java.lang.IllegalArgumentException","message":"n
> must be positive"}}
> Need to fix the report to give user hint about dead datanode.
--
This message was sent by Atlassian JIRA
(v6.1.5#6160)
