ShreyeshArangath commented on code in PR #1596:
URL: https://github.com/apache/auron/pull/1596#discussion_r2490975098
##########
.github/workflows/tpcds-reusable.yml:
##########
@@ -175,7 +175,7 @@ jobs:
UNIFFLE_NUMBER="${UNIFFLE_NUMBER#uniffle-}"
fi
- CMD="./auron-build.sh --pre --sparkver $SPARK_NUMBER --scalaver ${{
inputs.scalaver }}"
+ CMD="./auron-build.sh --pre --sparkver $SPARK_NUMBER --scalaver ${{
inputs.scalaver }} --skiptests false"
Review Comment:
I understand the concern, but in my opinion, even with a separate pipeline,
we’ll still be testing with different profiles in a format similar to the
framework already provided by the TPC-DS tests. Here’s why I don’t think this
is a major issue:
1. The current number of tests is quite low
2. All testing happens during the Build Auron JAR step in the pipeline
(picture). While this might slightly delay the start of TPC-DS queries, it
shouldn’t trigger false regression alarms
3. At this stage, I’m not sure it makes sense to introduce another set of n
jobs that essentially duplicate what the Build Auron JAR step already does
<img width="837" height="107" alt="image"
src="https://github.com/user-attachments/assets/ddb02f53-f300-4831-952c-826347956b5f";
/>
--
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]
