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-3294: [C++][Flight] Support Flight on Windows #4410
Conversation
dc6b821
to
7da51ae
Compare
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.
Thanks for picking this up. I'll review again when the PR is finished
1328785
to
f44f5f9
Compare
Should be ok now. Of course, a last-minute failure on CI is not out of question. |
f44f5f9
to
f394709
Compare
@lidavidm Any concerns here? |
f394709
to
1db74d7
Compare
1db74d7
to
bd4979b
Compare
Nice green build. I'm having a final look through |
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. thanks for doing this!
rd /s /q C:\OpenSSL-v11-Win32 | ||
rd /s /q C:\OpenSSL-v11-Win64 | ||
rd /s /q C:\OpenSSL-v111-Win32 | ||
rd /s /q C:\OpenSSL-v111-Win64 |
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.
Yuck
No description provided.