[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=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.

Reply via email to