[ 
https://issues.apache.org/jira/browse/HDFS-14121?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16721407#comment-16721407
 ] 

Hudson commented on HDFS-14121:
-------------------------------

SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #15607 (See 
[https://builds.apache.org/job/Hadoop-trunk-Commit/15607/])
HDFS-14121. Log message about the old hosts file format is misleading 
(templedf: rev aa1285989092bc253c45b7a83acec2e9bce2c5dc)
* (edit) 
hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/util/CombinedHostsFileReader.java


> Log message about the old hosts file format is misleading
> ---------------------------------------------------------
>
>                 Key: HDFS-14121
>                 URL: https://issues.apache.org/jira/browse/HDFS-14121
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>    Affects Versions: 3.0.0
>            Reporter: Daniel Templeton
>            Assignee: Zsolt Venczel
>            Priority: Major
>         Attachments: HDFS-14121.01.patch, HDFS-14121.02.patch
>
>
> In {{CombinedHostsFileReader.readFile()}} we have the following:
> {code}      LOG.warn("{} has invalid JSON format." +
>           "Try the old format without top-level token defined.", 
> hostsFile);{code}
> That message is trying to say that we tried parsing the hosts file as a 
> well-formed JSON file and failed, so we're going to try again assuming that 
> it's in the old badly-formed format.  What it actually says is that the hosts 
> fie is bad, and the admin should try switching to the old format.  Those are 
> two very different things.
> While were in there, we should refactor the logging so that instead of 
> reporting that we're going to try using a different parser (who the heck 
> cares?), we report that the we had to use the old parser to successfully 
> parse the hosts file.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to