-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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-10013: [FlightRPC][C++] fix setting generic client options #8196
Conversation
MacOS test flakes due to the filesystem issue; there are new Flight test failures on Windows in Python. |
d68c341
to
690a76d
Compare
On Windows, gRPC is apparently corrupting ServerContext::peer(), causing a test failure... |
d4b3f8e
to
1bff809
Compare
I'm going to hold this until the AWS SDK issues are resolved. The patch here should fix the MacOS tests and works around the Windows issues. |
1bff809
to
384f9e5
Compare
384f9e5
to
aaadb40
Compare
CC @pitrou, this will finally let AppVeyor pass again :) |
Wow, did you report the |
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
Not yet - I need to reproduce it in a VM first (probably with just base gRPC instead of trying to set up Arrow). |
Some gRPC version upgrade meant that for settings that were present twice, the first value was taken instead of the last value. This changes how we set default options to avoid this issue. Closes apache#8196 from lidavidm/arrow-10013 Authored-by: David Li <li.davidm96@gmail.com> Signed-off-by: Antoine Pitrou <antoine@python.org>
Some gRPC version upgrade meant that for settings that were present twice, the first value was taken instead of the last value. This changes how we set default options to avoid this issue. Closes apache#8196 from lidavidm/arrow-10013 Authored-by: David Li <li.davidm96@gmail.com> Signed-off-by: Antoine Pitrou <antoine@python.org>
Some gRPC version upgrade meant that for settings that were present twice, the first value was taken instead of the last value. This changes how we set default options to avoid this issue.