-
Notifications
You must be signed in to change notification settings - Fork 889
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create separate module for Protobuf files #691
Labels
Comments
ivankelly
added a commit
to ivankelly/bookkeeper
that referenced
this issue
Nov 9, 2017
Upcoming changes are using GRpc in conjunction with protobuf. GRpc generates code that created deprecation warnings when compiled with java 8, so this change moves all generated code out to another module, so that we don't have to turn off -Werror for all code. In any case, at some point we should split the bookkeeper client out from the server module, at which point we would need the definitions on a common place.
philipsu522
pushed a commit
to philipsu522/bookkeeper
that referenced
this issue
Dec 8, 2017
Upcoming changes are using GRpc in conjunction with protobuf. GRpc generates code that created deprecation warnings when compiled with java 8, so this change moves all generated code out to another module, so that we don't have to turn off -Werror for all code. In any case, at some point we should split the bookkeeper client out from the server module, at which point we would need the definitions on a common place. Author: Ivan Kelly <ivank@apache.org> Reviewers: Enrico Olivelli <eolivelli@gmail.com>, Jia Zhai <None>, Sijie Guo <sijie@apache.org> This closes apache#711 from ivankelly/proto-module, closes apache#691
aojea
pushed a commit
to aojea/bookkeeper
that referenced
this issue
Dec 16, 2017
Upcoming changes are using GRpc in conjunction with protobuf. GRpc generates code that created deprecation warnings when compiled with java 8, so this change moves all generated code out to another module, so that we don't have to turn off -Werror for all code. In any case, at some point we should split the bookkeeper client out from the server module, at which point we would need the definitions on a common place. Author: Ivan Kelly <ivank@apache.org> Reviewers: Enrico Olivelli <eolivelli@gmail.com>, Jia Zhai <None>, Sijie Guo <sijie@apache.org> This closes apache#711 from ivankelly/proto-module, closes apache#691
ivankelly
added a commit
to ivankelly/bookkeeper
that referenced
this issue
Mar 27, 2018
Upcoming changes are using GRpc in conjunction with protobuf. GRpc generates code that created deprecation warnings when compiled with java 8, so this change moves all generated code out to another module, so that we don't have to turn off -Werror for all code. In any case, at some point we should split the bookkeeper client out from the server module, at which point we would need the definitions on a common place.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Protobuf compiler has some bug and created codes with warning
Seen in #670
We can move protobuf related stuff to a separate module.
Thinking about separating client-side code from bookie-side code this will be a necessary step
The text was updated successfully, but these errors were encountered: