[AURON #1769][CI] Improve auron-it build #1932
Open
+22
−208
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
yew1eb
force-pushed
the
AURON_1769_ci_auron_it
branch
4 times, most recently
from
70397df to
dea1888
Compare
yew1eb
force-pushed
the
AURON_1769_ci_auron_it
branch
from
dea1888 to
c942c86
Compare
opt-level = 1 in ci
yew1eb
force-pushed
the
AURON_1769_ci_auron_it
branch
from
c942c86 to
4cfc418
Compare
Contributor
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.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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?