-
Notifications
You must be signed in to change notification settings - Fork 28.7k
[WIP][SPARK-32833][SQL] JDBC V2 Datasource aggregate push down #32061
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
Conversation
cc @huaxingao |
568d726
to
be4e673
Compare
be4e673
to
75f8fdc
Compare
ok to test. |
add to whitelist. |
Kubernetes integration test starting |
Kubernetes integration test status failure |
ok to test. |
Kubernetes integration test starting |
Kubernetes integration test status failure |
Test build #136977 has finished for PR 32061 at commit
|
Test build #136991 has finished for PR 32061 at commit
|
Kubernetes integration test starting |
Kubernetes integration test status failure |
We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable. |
What changes were proposed in this pull request?
This PR refactor #29695, it has some limitations in terms of implementation and abstraction:
count
AVG
in case of multiple partitions, and actually the current implementation is wrong even in single partition.sum(if())
.see the discussion here, The SPIP PDF could be viewed here.
Why are the changes needed?
for better performance
Does this PR introduce any user-facing change?
No
How was this patch tested?
using tests from #29695