-
Notifications
You must be signed in to change notification settings - Fork 838
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
Only run integration tests when arrow
changes
#2152
Only run integration tests when arrow
changes
#2152
Conversation
arrow
changes
|
||
jobs: | ||
|
||
integration: | ||
name: Integration Test | ||
name: Archery test With other arrows |
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.
renamed this to be more specific
Codecov Report
@@ Coverage Diff @@
## master #2152 +/- ##
=======================================
Coverage 82.86% 82.86%
=======================================
Files 237 237
Lines 61296 61296
=======================================
+ Hits 50792 50793 +1
+ Misses 10504 10503 -1
Help us with your feedback. Take ten seconds to tell us how you rate us. |
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.
This looks good to me.
Benchmark runs are scheduled for baseline = aa5c849 and contender = 7746e7d. 7746e7d is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
Which issue does this PR close?
re #2149
Rationale for this change
These actions test the integration of the
arrow
crate, yet they run on all PRs (includingparquet
changes and now alsoobject_store
changes) and they take significant time (32 minutes), often the longest test in the suiteWhat changes are included in this PR?
arrow
or.github
changeAre there any user-facing changes?
no
Testing:
.github
, in this PR): https://github.com/apache/arrow-rs/runs/7483623018?check_suite_focus=truearrow
): I need to merge this PR to testparquet
): I need to merge this PR to test