You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@bookkeeper.apache.org by "Ivan Kelly (JIRA)" <ji...@apache.org> on 2014/03/07 17:34:43 UTC
[jira] [Commented] (BOOKKEEPER-582) Make bookie and client use
protobuf for requests (non-wire part)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-582?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13924028#comment-13924028 ]
Ivan Kelly commented on BOOKKEEPER-582:
---------------------------------------
One thing we have to be aware of here is that by using enums, we are committing to losings client->server compatibility whenever we upgrade the protocol. To me, this doesn't seem to be such a problem, as we tend to mandate that servers are upgraded before clients, but I think it's something we should get a little bit more consensus on. [~rakeshr] [~umamaheswararao] Is server client incompatibility ok for you guys as long as the server upgrades first?
> 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)