[
https://issues.apache.org/jira/browse/BOOKKEEPER-582?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13944144#comment-13944144
]
Rakesh R commented on BOOKKEEPER-582:
-------------------------------------
First I'd like to thank you guys([~hustlmsp], [~ikelly] and others) for the
good work and the progress.
Hi Ivan, sorry for the delay in replay and I also feel we can push the SSL
patches.
bq.Is server client incompatibility ok for you guys as long as the server
upgrades first?
Today I've gone through the diff present in the review board. Very first look,
its ok to me and overall BK server code looks fine and taken care the
compatibility part well. Also [~fpj] has reviewed the code.
Like you mentioned while upgrading it should requires to upgrade bk server
first and then clients(HDFS et.). This way it would be able to do the upgrade
without having to shutdown the entire services(both clients as well as
server). Anyway I'll have a closer look again and will come back in a day or
two.
> Make bookie and client use protobuf for requests (non-wire part)
> ----------------------------------------------------------------
>
> Key: BOOKKEEPER-582
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-582
> Project: Bookkeeper
> Issue Type: Sub-task
> Components: bookkeeper-client, bookkeeper-server
> Reporter: Ivan Kelly
> Assignee: Aniruddha
> Fix For: 4.3.0
>
> Attachments:
> 0002-BOOKKEEPER-582-Make-bookie-and-client-use-protobuf-f.patch,
> 0002-BOOKKEEPER-582-Make-bookie-and-client-use-protobuf-f.patch,
> 0002-BOOKKEEPER-582-Make-bookie-and-client-use-protobuf-f.patch,
> 0003-BOOKKEEPER-582-Make-bookie-and-client-use-protobuf-f.patch,
> BOOKKEEPER-582.diff
>
>
> Make the client and the bookie use protobufs internally. This is the first
> step to using protobufs on the wire, but for the moment, BookieRequestHandler
> decodes the old wire protocol into the protobuf messages. Once this is in,
> enabling on the wire will be very simple, and the old manual serialization
> can be made "legacy" (still supported, but deprecated).
--
This message was sent by Atlassian JIRA
(v6.2#6252)
