[jira] Commented: (ZOOKEEPER-588) remove unnecessary/annoying log of tostring error in Request.toString()

2009-12-14 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12790088#action_12790088
 ] 

Hudson commented on ZOOKEEPER-588:
--

Integrated in ZooKeeper-trunk #625 (See 
[http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/625/])


 remove unnecessary/annoying log of tostring error in Request.toString()
 ---

 Key: ZOOKEEPER-588
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-588
 Project: Zookeeper
  Issue Type: Bug
  Components: server
Affects Versions: 3.2.1
Reporter: Patrick Hunt
Assignee: Patrick Hunt
Priority: Minor
 Fix For: 3.3.0

 Attachments: ZOOKEEPER-588.patch


 Why are we logging this? It's unnecessary and just annoying afaict. We should 
 remove it entirely.
 2009-11-18 05:37:29,312 WARN org.apache.zookeeper.server.Request: Ignoring 
 exception during toString
 java.nio.BufferUnderflowException
   at java.nio.HeapByteBuffer.get(HeapByteBuffer.java:127)
   at java.nio.ByteBuffer.get(ByteBuffer.java:675)
   at org.apache.zookeeper.server.Request.toString(Request.java:199)
   at java.lang.String.valueOf(String.java:2827)
   at java.lang.StringBuilder.append(StringBuilder.java:115)
   at 
 org.apache.zookeeper.server.quorum.CommitProcessor.processRequest(CommitProcessor.java:167)
   at 
 org.apache.zookeeper.server.quorum.FollowerRequestProcessor.run(FollowerRequestProcessor.java:68)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (ZOOKEEPER-588) remove unnecessary/annoying log of tostring error in Request.toString()

2009-11-24 Thread Patrick Hunt (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12782057#action_12782057
 ] 

Patrick Hunt commented on ZOOKEEPER-588:


I always do that for multi-line conditional, I think it makes it easier to find 
the block bounds

 remove unnecessary/annoying log of tostring error in Request.toString()
 ---

 Key: ZOOKEEPER-588
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-588
 Project: Zookeeper
  Issue Type: Bug
  Components: server
Affects Versions: 3.2.1
Reporter: Patrick Hunt
Assignee: Patrick Hunt
Priority: Minor
 Fix For: 3.3.0

 Attachments: ZOOKEEPER-588.patch


 Why are we logging this? It's unnecessary and just annoying afaict. We should 
 remove it entirely.
 2009-11-18 05:37:29,312 WARN org.apache.zookeeper.server.Request: Ignoring 
 exception during toString
 java.nio.BufferUnderflowException
   at java.nio.HeapByteBuffer.get(HeapByteBuffer.java:127)
   at java.nio.ByteBuffer.get(ByteBuffer.java:675)
   at org.apache.zookeeper.server.Request.toString(Request.java:199)
   at java.lang.String.valueOf(String.java:2827)
   at java.lang.StringBuilder.append(StringBuilder.java:115)
   at 
 org.apache.zookeeper.server.quorum.CommitProcessor.processRequest(CommitProcessor.java:167)
   at 
 org.apache.zookeeper.server.quorum.FollowerRequestProcessor.run(FollowerRequestProcessor.java:68)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.