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

Enrico Olivelli commented on BOOKKEEPER-937:
--------------------------------------------

I'm +1 in upgrading to 2.6 if it is compatibile.

.java files generated by protobuf compiler actually need to be committed as 
they are never re-generated during builds

maybe it would be better to submit a Github PR using the new patch submission 
flow




> Upgrade protobuf to 2.6
> -----------------------
>
>                 Key: BOOKKEEPER-937
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-937
>             Project: Bookkeeper
>          Issue Type: Improvement
>          Components: bookkeeper-server
>    Affects Versions: 4.5.0
>            Reporter: Andrey Yegorov
>            Priority: Minor
>         Attachments: 0001-upgraded-protobuf-to-2.6.1.patch
>
>
> I had to update protobuf definition for some internal experiments and found 
> that working with protobuf 2.4 is rather inconvenient. It cannot be installed 
> with brew on mac and building it on mac always result is build errors hence 
> leaves an option of switching to linux to run protoc.
> I decided to upgrade to 2.6 instead. It is compatible with 2.4 on wire and 
> shaded so should not create any problems. All tests passed.
> Please ignore changes in java files in attached patch during review; these 
> are auto-generated.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to