Skip to content

CI: Temporarily build pandas with N_JOBS=1 to avoid flakiness #47341

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

Merged
merged 6 commits into from
Jun 15, 2022

Conversation

mroeschke
Copy link
Member

xref: #47305

@mroeschke mroeschke added Build Library building on various platforms CI Continuous Integration labels Jun 13, 2022
@mroeschke mroeschke marked this pull request as ready for review June 13, 2022 23:12
@mroeschke mroeschke changed the title CI: Temporary build pandas with N_JOBS=1 to avoid flakiness CI: Temporarily build pandas with N_JOBS=1 to avoid flakiness Jun 13, 2022
@jbrockmendel
Copy link
Member

Does this affect the CI speed?

@mroeschke
Copy link
Member Author

Looks like the build step goes from ~4-6 min to ~8-10 min. IMO it's an okay trade off to prevent flakiness.

Plus caching these builds might be in the works soon #46643

@mroeschke mroeschke added this to the 1.5 milestone Jun 14, 2022
@jbrockmendel
Copy link
Member

Works for me

@mroeschke
Copy link
Member Author

The ImportError hasn't show up when only using 1 worker after a few merges of main, so I think this temporarily solves the flakiness. Merging.

@mroeschke mroeschke merged commit 4489725 into pandas-dev:main Jun 15, 2022
@mroeschke mroeschke deleted the ci/tmp/parallel branch June 15, 2022 22:12
yehoshuadimarsky pushed a commit to yehoshuadimarsky/pandas that referenced this pull request Jul 13, 2022
…-dev#47341)

* CI: Temporary build pandas with N_JOBS=1 to avoid flakiness

* Check again
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build Library building on various platforms CI Continuous Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants