Skip to content

Conversation

@v-jizhang
Copy link
Contributor

@v-jizhang v-jizhang commented May 24, 2023

Split tests in e2e job for them to be run in parallel. Without changing our test code, circle ci is able to run each test in its own container in parallel without interfering with each other.
This cuts the overall e2e execution time from 1.5 hours to around 30 minutes with parallelism set to 8. Adding the build time the entire process could finish in about 45 minutes.

We also separate Spark tests in a different run so we can clearly see if the failures is from spark or elsewhere.

Test plan - Make sure all the tests pass.

== NO RELEASE NOTE ==

@v-jizhang v-jizhang requested a review from a team as a code owner May 24, 2023 15:40
@v-jizhang v-jizhang requested a review from ajaygeorge May 24, 2023 15:40
@v-jizhang v-jizhang marked this pull request as draft May 24, 2023 21:57
@v-jizhang v-jizhang marked this pull request as ready for review May 25, 2023 01:36
@v-jizhang
Copy link
Contributor Author

@prestobot kick off tests

@mshang816 mshang816 changed the title [native] Migrate presto-native-execution CI to CircleCI[third phase] [native] Split tests in e2e job for them to run in parallel May 25, 2023
@mshang816 mshang816 changed the title [native] Split tests in e2e job for them to run in parallel [native] Split tests in e2e job for them to be run in parallel May 25, 2023
@v-jizhang
Copy link
Contributor Author

@prestobot kick off tests

@v-jizhang v-jizhang force-pushed the circle-ci3 branch 4 times, most recently from ef48788 to 8a24b12 Compare May 29, 2023 15:00
@v-jizhang
Copy link
Contributor Author

@prestobot kick off tests

@v-jizhang v-jizhang force-pushed the circle-ci3 branch 5 times, most recently from cf2351a to 2a7a7c5 Compare May 31, 2023 18:35
Split E2E tests and run them in parallel. Also create a separate
run for Spark tests.

Co-authored-by: Michael Shang <mikesh@fb.com>
Copy link
Contributor

@mshang816 mshang816 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants