You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2015/05/06 13:48:00 UTC
[jira] [Commented] (ZOOKEEPER-2176) Unclear error message should be
info not error
[ 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)