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

Hudson commented on BOOKKEEPER-576:
-----------------------------------

Integrated in bookkeeper-trunk #141 (See 
[https://builds.apache.org/job/bookkeeper-trunk/141/])
    BOOKKEEPER-576: Bookie client should use netty Decoder/Encoder (ivank via 
sijie) (Revision 1457592)

     Result = SUCCESS
sijie : 
Files : 
* /zookeeper/bookkeeper/trunk/CHANGES.txt
* 
/zookeeper/bookkeeper/trunk/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/BookieNettyServer.java
* 
/zookeeper/bookkeeper/trunk/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/BookieProtoEncoding.java
* 
/zookeeper/bookkeeper/trunk/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/BookieProtocol.java
* 
/zookeeper/bookkeeper/trunk/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/BookieRequestHandler.java
* 
/zookeeper/bookkeeper/trunk/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/PerChannelBookieClient.java
* 
/zookeeper/bookkeeper/trunk/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/ResponseBuilder.java
* 
/zookeeper/bookkeeper/trunk/bookkeeper-server/src/test/java/org/apache/bookkeeper/proto/TestProtoVersions.java

                
> Bookie client should use netty Decoder/Encoder
> ----------------------------------------------
>
>                 Key: BOOKKEEPER-576
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-576
>             Project: Bookkeeper
>          Issue Type: Sub-task
>            Reporter: Ivan Kelly
>            Assignee: Ivan Kelly
>             Fix For: 4.3.0
>
>         Attachments: 
> 0002-BOOKKEEPER-576-Bookie-client-should-use-netty-Decode.patch
>
>
> Current it encodes and decodes in an adhoc manner in PerChannelBookieClient. 
> It would be better to have the request encoding and decoding in the same 
> place and it should also be done with a OneToOne*coder like is done with the 
> server. 
> This will make it easier to add a versioning decoder, which will make it much 
> easier to extend the bookie protocol.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to