-
Notifications
You must be signed in to change notification settings - Fork 168
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
chore: Enable Comet shuffle with AQE coalesce partitions #651
Conversation
Some Spark tests need to be updated. Marked this as a draft for now. |
bce36c5
to
5e0dd3b
Compare
All Comet and TPCDS, TPCH tests are passed. Next, I will fix Spark SQL tests. |
17914f6
to
6a9b55b
Compare
4527010
to
afe1176
Compare
769677f
to
c406e40
Compare
c406e40
to
5cbf4f4
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #651 +/- ##
============================================
+ Coverage 33.94% 33.99% +0.05%
+ Complexity 874 858 -16
============================================
Files 112 112
Lines 42916 42875 -41
Branches 9464 9461 -3
============================================
+ Hits 14567 14576 +9
+ Misses 25379 25304 -75
- Partials 2970 2995 +25 ☔ View full report in Codecov by Sentry. |
b247bb8
to
b960536
Compare
6cc5438
to
b636f58
Compare
55191f7
to
0e01818
Compare
For the negative refcount issue reported by Java Arrow, I cannot reproduce it locally or even on internal CI (different platform other than ubuntu). I'm not sure if it is related to the ubuntu platform the GitHub Actions uses. Because GitHub Actions only supports ubuntu for linux, I cannot try other linux os. Currently I can only try to use older ubuntu (20.04) and see it helps. |
Close in favor of #834. |
Which issue does this PR close?
Closes #387.
Rationale for this change
What changes are included in this PR?
How are these changes tested?