Skip to content

Commit

Permalink
[native] Disable Spark E2E test CI job
Browse files Browse the repository at this point in the history
The Spark E2E tests have been causing problems since migration from
CircleCI to github actions. The latter includes more tests to be run that cause
an instability in the pipeline because they were previously not run consistently.
This resulted in them becoming flaky affecting the entire pipeline.

This PR explicitly runs tests from the previous CI as well as other selected
tests that are not flaky.

Excluded tests are
TestPrestoSparkNativeGeneralQueries
TestPrestoSparkNativeAggregations
TestPrestoSparkNativeJoinQueries
TestPrestoSparkNativeWindowQueries

Issues have been opened to address the various observed failures.
  • Loading branch information
czentgr committed Jan 15, 2025
1 parent 761ab56 commit c67359c
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions .github/workflows/prestocpp-linux-build-and-unit-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -235,6 +235,9 @@ jobs:
export TESTCLASSES="${TESTCLASSES},$test_class"
done
export TESTCLASSES=${TESTCLASSES#,}
# Override the possible test classes with test classes known to pass from the previous CI.
# A number of newly added tests are flaky and are disabled.
export TESTCLASSES=TestPrestoSparkExpressionCompiler,TestPrestoSparkNativeJoinQueries,TestPrestoSparkSqlFunctions,TestPrestoSparkNativeTpchQueries,TestPrestoSparkNativeArrayFunctionQueries,TestPrestoSparkNativeGeneralQueries
echo "TESTCLASSES = $TESTCLASSES"
mvn test \
${MAVEN_TEST} \
Expand Down

0 comments on commit c67359c

Please sign in to comment.