Skip to content

Disable test parallelization in TimeSeries tests #4757

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Disable test parallelization in TimeSeries tests #4757

wants to merge 1 commit into from

Conversation

sharwell
Copy link
Contributor

Extracted from #4569

@sharwell sharwell requested a review from a team as a code owner January 31, 2020 19:48
@sharwell sharwell added the test related to tests label Jan 31, 2020
@frank-dong-ms-zz
Copy link
Contributor

what is the purpose of disabling test parallelization in TS tests?

@sharwell
Copy link
Contributor Author

sharwell commented Jan 31, 2020 •
edited
Loading

I meant to make this one a draft PR. I'm not certain it's correct necessary but didn't want to lose track of the commit during a rebase.

@harishsk
Copy link
Contributor

Do we still need to keep this PR open?

@mstfbl mstfbl marked this pull request as draft April 21, 2020 03:33
@mstfbl
Copy link
Contributor

mstfbl commented Apr 21, 2020

Hi @sharwell , I've converted this PR to a draft PR as you said you meant to. Feel free to close the PR if you don't need it open anymore. Thanks.

@sharwell sharwell closed this Apr 21, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Mar 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
test related to tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@sharwell @frank-dong-ms-zz @harishsk @mstfbl