-
Notifications
You must be signed in to change notification settings - Fork 201
[AURON #1769][AIP-3]: Introduce auron-it for Enhanced CI Integration Testing #1802
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
9deb3e5 to
7028f93
Compare
7e20538 to
69e55ed
Compare
|
@cxzl25 When you have a moment, please help review as well. Summary of changes: 1. result check 2. plan check 3. regenerate golden files |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR introduces the auron-it module to enhance CI integration testing capabilities for the Auron project. It establishes infrastructure for TPC-DS benchmark testing and query plan validation.
Key changes:
- New
auron-itmodule with TPC-DS query plan stability testing framework - Added comprehensive shell scripts for running integration tests locally
- Modified build process to use
installinstead ofpackagefor better dependency management
Reviewed changes
Copilot reviewed 29 out of 226 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| dev/auron-it/src/main/resources/tpcds-plan-stability/spark-3.5/*.txt | Golden files containing expected query execution plans for TPC-DS queries (q22, q24b, q25, q26, q27, q28) |
| dev/auron-it/src/main/resources/log4j*.properties | Logging configuration files for integration testing |
| dev/auron-it/run-it.sh | Main script to execute integration tests with Spark |
| dev/auron-it/local-run-tpcds.sh | Helper script demonstrating TPC-DS benchmark execution |
| auron-build.sh | Updated Maven goals from package to install |
| .rat-excludes | Excluded TPC-DS test resources from Apache RAT license checks |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
fd8a757 to
14b6ddf
Compare
Which issue does this PR close?
Closes #1769
Rationale for this change
What changes are included in this PR?
Are there any user-facing changes?
How was this patch tested?