gh-yzou commented on code in PR #3641:
URL: https://github.com/apache/polaris/pull/3641#discussion_r2780724350
##########
plugins/spark/v3.5/regtests/run.sh:
##########
@@ -70,6 +70,21 @@ SPARK_VERSION="3.5.6"
SPARK_SHELL_OPTIONS=("PACKAGE" "JAR")
+# Define test suites to run
+# Each suite specifies: test_file:table_format:test_shortname
+declare -a TEST_SUITES=(
+ "spark_sql.sh:delta:spark_sql"
Review Comment:
How about let's enforce the test file name to the format like
xxx_<table_format>.sh, and have a separate folder to include all test src file
and reference file. Then we just need to list the folder to get all test files,
and extract the table format by parsing the file name. The benefit would be
easy to onboard new tests, and developer doesn't have to input a long string
when running single test (just the file name)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
