-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
GH-40153: [C++][Python] Fix test_gdb failures on 32-bit #40293
Conversation
@github-actions crossbow submit -g python |
This comment was marked as outdated.
This comment was marked as outdated.
@AlenkaF @jorisvandenbossche Does one of you want to comment on this? |
No comment on the code, looks good to me. |
Yes, it is. |
11012a9
to
d42ed96
Compare
@github-actions crossbow submit -g python |
Revision: d42ed96 Submitted crossbow builds: ursacomputing/crossbow @ actions-c765d70b22 |
Test failures are unrelated. |
After merging your PR, Conbench analyzed the 5 benchmarking runs that have been run so far on merge-commit 53e0c74. There were no benchmark performance regressions. 🎉 The full Conbench report has more details. |
…#40293) ### Rationale for this change `test_gdb.py` would fail on 32-bit platforms because the gdb extension errors out when a timestamp value is larger than the platform's time_t. ### What changes are included in this PR? 1. Catch `OverflowError` from the Python datetime module when trying to format a timestamp 2. Tweak the expected test results on 32-bit ### Are these changes tested? Yes. ### Are there any user-facing changes? No. * GitHub Issue: apache#40153 Authored-by: Antoine Pitrou <[email protected]> Signed-off-by: Antoine Pitrou <[email protected]>
…#40293) ### Rationale for this change `test_gdb.py` would fail on 32-bit platforms because the gdb extension errors out when a timestamp value is larger than the platform's time_t. ### What changes are included in this PR? 1. Catch `OverflowError` from the Python datetime module when trying to format a timestamp 2. Tweak the expected test results on 32-bit ### Are these changes tested? Yes. ### Are there any user-facing changes? No. * GitHub Issue: apache#40153 Authored-by: Antoine Pitrou <[email protected]> Signed-off-by: Antoine Pitrou <[email protected]>
Rationale for this change
test_gdb.py
would fail on 32-bit platforms because the gdb extension errors out when a timestamp value is larger than the platform's time_t.What changes are included in this PR?
OverflowError
from the Python datetime module when trying to format a timestampAre these changes tested?
Yes.
Are there any user-facing changes?
No.