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
DL-205: Remove StatusCode dependency on DLException #131
Conversation
- move proxy related class from protocol to proxy-protocol, changed client and service to proxy-client and proxy-service
Build finished. |
1 similar comment
Build finished. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1. LGTM.
Thanks for this work.
@leighst do you mind reviewing this as well? in this change, I try to separate the protocols used for core from the protocols used by proxy, also rename the modules to make them clear. |
looks good. i guess the choice here is to make it like bk, to some extent. personally i think having a pojo StatusCode object would be better than passing around ints which can be easily misinterpreted. but i dont have a strong opinion. |
Currently finagle heavily depends on an out-of-dated version - libthrift 5.0. Proxy modules (client, server) depend on this version, however the core library doesn't really depend on libthrift. This change is to change libthrift to 0.9.* in distributedlog-core and shade it to avoid it conflict with the version used by finagle. This change is based on #131 . The main change is at gitsha [6e58786](6e58786) Author: Sijie Guo <sijie@apache.org> Reviewers: Jia Zhai <None>, Leigh Stewart <lstewart@apache.org> Closes #132 from sijie/change_thrift_for_core_module
Remove StatusCode from exceptions. Use integer as exception codes.
Also re-organize the modules:
distributedlog-protocol (for core structures) and distributedlog-core (for core library).
proxy: distributedlog-proxy-protocol (new module for thrift generated protocol), distributedlog-proxy-client (proxy client) and distributedlog-proxy-server (proxy server)
benchmark & tutorials.