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
ARROW-10710: [Rust] Revert tokio upgrade, go back to 0.2 #8769
Conversation
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.
LGTM
FYI @jorgecarleitao |
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.
I also found a regression related to this update in datafusion-python
on which the process hangs.
This reverts commit 4192891.
I am waiting for CI to finish running and then I will merge this in |
CI is green -- merging this in! |
## Changes This PR backs out the version upgrade to tokio done in apache#8697 and also partly by @andygrove in apache#8734 (see comment apache#8697 (comment) by @rdettai for more details). ## Rationale: It seems as if the new version of tokio (0.3) has some fairly substantial changes, and some of the higher level libraries built on top of tokio that are used in the flight implementation do not yet support version 0.3 of tokio. This results in parts of the code (flight server) using the older version of tokio and parts of the code (datafusion) using newer versions of tokio which causes ... issues. This issue can be seen in the flight example linked in https://issues.apache.org/jira/browse/ARROW-10710 and I saw it when I tried to upgrade to latest arrow in https://github.com/influxdata/influxdb_iox For example, https://crates.io/crates/tonic appears to require tokio < 0.3 <img width="350" alt="Screen Shot 2020-11-25 at 7 10 21 AM" src="https://user-images.githubusercontent.com/490673/100227458-2eabbb00-2eef-11eb-82a8-eb11dad03bc2.png"> cc @Dandandan and @nevi-me Closes apache#8769 from alamb/alamb/ARROW-10710-fix-fligh Authored-by: Andrew Lamb <andrew@nerdnetworks.org> Signed-off-by: Andrew Lamb <andrew@nerdnetworks.org>
Changes
This PR backs out the version upgrade to tokio done in #8697 and also partly by @andygrove in #8734 (see comment #8697 (comment) by @rdettai for more details).
Rationale:
It seems as if the new version of tokio (0.3) has some fairly substantial changes, and some of the higher level libraries built on top of tokio that are used in the flight implementation do not yet support version 0.3 of tokio. This results in parts of the code (flight server) using the older version of tokio and parts of the code (datafusion) using newer versions of tokio which causes ... issues.
This issue can be seen in the flight example linked in https://issues.apache.org/jira/browse/ARROW-10710 and I saw it when I tried to upgrade to latest arrow in https://github.com/influxdata/influxdb_iox
For example, https://crates.io/crates/tonic appears to require tokio < 0.3
cc @Dandandan and @nevi-me