-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[zeppelin-3639] Add Ipython interpreter prerequisite check for protobuf #3084
Conversation
LGTM |
Would be nice to check freeze output for tornado version. |
@Tagar thanks for suggestion, but I think it's better not to include this check for tornado in this function since it's not really pre-requisite check. if we add it, then we need to keep track of it, and restore back when the issue with tornado is fixed. |
that makes sense. thanks @khalidhuseynov |
This is to protobuf prerequisite check when starting ipython kernel. This is required since from grpcio 1.12.0 it's independent from protobuf [ Improvement ] * [x] - add protobuf prerequisite check * [x] - add to ipython docs https://issues.apache.org/jira/browse/ZEPPELIN-3639 1. remove grpcio and protobuf 2. install latest grpcio and try to run ipython interpreter - should fail 3. then install protobuf and try again - should work * Does the licenses files need update? * Is there breaking changes for older versions? * Does this needs documentation? Author: Khalid Huseynov <khalidhnv@gmail.com> Closes #3084 from khalidhuseynov/fix/ZEPPELIN-3639 and squashes the following commits: 066e061 [Khalid Huseynov] fix ipython documentation 58c0689 [Khalid Huseynov] add protobuf prerequsite check (cherry picked from commit c95d896) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for? This is to protobuf prerequisite check when starting ipython kernel. This is required since from grpcio 1.12.0 it's independent from protobuf ### What type of PR is it? [ Improvement ] ### Todos * [x] - add protobuf prerequisite check * [x] - add to ipython docs ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-3639 ### How should this be tested? 1. remove grpcio and protobuf 2. install latest grpcio and try to run ipython interpreter - should fail 3. then install protobuf and try again - should work ### Screenshots (if appropriate) ### Questions: * Does the licenses files need update? * Is there breaking changes for older versions? * Does this needs documentation? Author: Khalid Huseynov <khalidhnv@gmail.com> Closes apache#3084 from khalidhuseynov/fix/ZEPPELIN-3639 and squashes the following commits: 066e061 [Khalid Huseynov] fix ipython documentation 58c0689 [Khalid Huseynov] add protobuf prerequsite check
This is to protobuf prerequisite check when starting ipython kernel. This is required since from grpcio 1.12.0 it's independent from protobuf [ Improvement ] * [x] - add protobuf prerequisite check * [x] - add to ipython docs https://issues.apache.org/jira/browse/ZEPPELIN-3639 1. remove grpcio and protobuf 2. install latest grpcio and try to run ipython interpreter - should fail 3. then install protobuf and try again - should work * Does the licenses files need update? * Is there breaking changes for older versions? * Does this needs documentation? Author: Khalid Huseynov <khalidhnv@gmail.com> Closes apache#3084 from khalidhuseynov/fix/ZEPPELIN-3639 and squashes the following commits: 066e061 [Khalid Huseynov] fix ipython documentation 58c0689 [Khalid Huseynov] add protobuf prerequsite check (cherry picked from commit c95d896) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
What is this PR for?
This is to protobuf prerequisite check when starting ipython kernel. This is required since from grpcio 1.12.0 it's independent from protobuf
What type of PR is it?
[ Improvement ]
Todos
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-3639
How should this be tested?
Screenshots (if appropriate)
Questions: