Skip to content
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

[C++][CI] Flight test crashes on MinGW #28845

Closed
asfimport opened this issue Jun 22, 2021 · 8 comments
Closed

[C++][CI] Flight test crashes on MinGW #28845

asfimport opened this issue Jun 22, 2021 · 8 comments

Comments

@asfimport
Copy link
Collaborator

asfimport commented Jun 22, 2021

On the "MinGW 64 GLib & Ruby" build:

https://github.com/apache/arrow/pull/10566/checks?check_run_id=2883812077

On the "MinGW 64 C++" build:

https://github.com/apache/arrow/pull/10566/checks?check_run_id=2883812286

Reporter: Antoine Pitrou / @pitrou
Assignee: David Li / @lidavidm

Related issues:

Note: This issue was originally created as ARROW-13145. Please see the migration documentation for further details.

@asfimport
Copy link
Collaborator Author

Antoine Pitrou / @pitrou:
cc @lidavidm , @kou

@asfimport
Copy link
Collaborator Author

David Li / @lidavidm:
I just saw this and filed ARROW-13144 as well, I'll cancel that one.

@asfimport
Copy link
Collaborator Author

David Li / @lidavidm:
I can reproduce it, but there's not much to debug. It looks like it crashes in gRPC (but the debugger can't get a backtrace anymore, and I can't build the bundled gRPC).

I noticed that the last passing run installed mingw-w64-x86_64-grpc-1.35.0-2, while the failing runs install mingw-w64-x86_64-grpc-1.35.0-3 - I wonder what changed there.

@asfimport
Copy link
Collaborator Author

David Li / @lidavidm:
It looks like the revision adjusted some LDFLAGS (see msys2/MINGW-packages@5f40e4c) - I'm going to build the package from scratch (and with debug symbols) and see if that takes me anywhere.

@asfimport
Copy link
Collaborator Author

David Li / @lidavidm:
I can confirm rebuilding with the LDFLAGS adjusted fixes it - so maybe that shouldn't have been taken out. (Even with debug info, though, there's no backtrace - seems whatever gRPC does, it corrupts the stack.)

@asfimport
Copy link
Collaborator Author

David Li / @lidavidm:
I submitted msys2/MINGW-packages#8984 and will probably follow up with a PR depending on how the maintainers react.

@asfimport
Copy link
Collaborator Author

David Li / @lidavidm:
Upstream released a new package revision that should fix the issue. I'll close this once I can confirm our CI passes again.

@asfimport
Copy link
Collaborator Author

David Li / @lidavidm:
The new revision fixed the issue (e.g. see https://github.com/apache/arrow/runs/2904412656).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment