[AURON #1769][CI] Improve auron-it build#1932
Conversation
yew1eb
force-pushed
the
AURON_1769_ci_auron_it
branch
6 times, most recently
from
c942c86 to
4cfc418
Compare
There was a problem hiding this comment.
Pull request overview
This PR improves the auron-it (integration test) build process by incorporating version and Scala variant information into the artifact names, making them consistent with the main Auron JAR naming scheme.
Changes:
- Updated auron-it artifact naming to include Spark version and Scala version
- Modified CI workflow to build and upload auron-it JAR as a separate artifact instead of relying on Maven repository
- Updated run-it.sh script to support the new naming pattern
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| dev/auron-it/pom.xml | Changed artifactId to include ${shimName}_${scalaVersion} for version-specific builds |
| dev/auron-it/run-it.sh | Updated AURON_IT_JAR path to include SPARK_VERSION and SCALA_VERSION in the jar filename |
| dev/auron-it/dependency-reduced-pom.xml | Removed generated file (created by maven-shade-plugin) |
| .github/workflows/tpcds-reusable.yml | Added build step for auron-it, changed artifact upload/download to use specific jar instead of Maven repository artifacts |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
ShreyeshArangath
left a comment
ShreyeshArangath
left a comment
There was a problem hiding this comment.
minor comments on using auron-it instead of Auron it for readability
yew1eb
force-pushed
the
AURON_1769_ci_auron_it
branch
2 times, most recently
from
3535e2e to
75f9cb6
Compare
opt-level = 1 in ci
yew1eb
force-pushed
the
AURON_1769_ci_auron_it
branch
from
75f9cb6 to
1e2d7c4
Compare
|
cc @richox |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Which issue does this PR close?
Closes #1802
Rationale for this change
What changes are included in this PR?
Are there any user-facing changes?
How was this patch tested?