zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ZOOKEEPER-2176) Unclear error message should be info not error
Date Wed, 06 May 2015 11:48:00 GMT

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

Hudson commented on ZOOKEEPER-2176:
-----------------------------------

FAILURE: Integrated in ZooKeeper-trunk #2683 (See [https://builds.apache.org/job/ZooKeeper-trunk/2683/])
ZOOKEEPER-2176 Unclear error message should be info not error (rgs via hdeng) (hdeng: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1677865)
* /zookeeper/trunk/CHANGES.txt
* /zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java


> Unclear error message should be info not error
> ----------------------------------------------
>
>                 Key: ZOOKEEPER-2176
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2176
>             Project: ZooKeeper
>          Issue Type: Improvement
>          Components: quorum
>    Affects Versions: 3.5.0
>            Reporter: Raul Gutierrez Segales
>            Assignee: Raul Gutierrez Segales
>             Fix For: 3.5.1, 3.6.0
>
>         Attachments: ZOOKEEPER-2176.patch
>
>
> Hi [~shralex],
> Looking at the CI output of ZOOKEEPER-2163 I see this:
> {noformat}
>  [exec]     [junit] 2015-04-17 17:36:23,750 [myid:] - ERROR [QuorumPeer[myid=4](plain=/0:0:0:0:0:0:0:0:11235)(secure=disabled):QuorumPeer@1394]
- writeToDisk == true but configFilename == null
> {noformat}
> Though looking at QuorumPeer#setQuorumVerifier I see:
> {noformat}
>             if (configFilename != null) {
>                 try {
>                     String dynamicConfigFilename = makeDynamicConfigFilename(
>                             qv.getVersion());
>                     QuorumPeerConfig.writeDynamicConfig(
>                             dynamicConfigFilename, qv, false);
>                     QuorumPeerConfig.editStaticConfig(configFilename,
>                             dynamicConfigFilename,
>                             needEraseClientInfoFromStaticConfig());
>                 } catch (IOException e) {
>                     LOG.error("Error closing file: ", e.getMessage());
>                 }
>             } else {
>                 LOG.error("writeToDisk == true but configFilename == null");
>             }
> {noformat}
> there's no proper error handling so I guess maybe we should just make it a warning? Thoughts?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message