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
Fix Horovod pyarrow IndexError: list index out of range #3274
Conversation
1c69a9a
to
8354368
Compare
@tgaddair I created a new PR for #2193 , could you help review ? It should address the issue mentioned here: #3255 (comment) |
Unit Test Results (with flaky tests) 876 files - 2 876 suites - 2 6h 28m 31s ⏱️ - 1m 45s Results for commit 2b88126. ± Comparison against base commit d395a88. ♻️ This comment has been updated with latest results. |
Signed-off-by: Weichen Xu <weichen.xu@databricks.com>
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.
Can you rebase to / merge latest master please? Then I will run the Spark tests on this branch (those that are currently disabled) to check they do not break on this fix.
@WeichenXu123 I am running this branch with all Spark tests reactivated (#3263) in #3283. |
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.
All tests are green: #3283
Checklist before submitting
Description
Fixes #2193 .
Fix Horovod pyarrow IndexError: list index out of range
Review process to land