-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Add Druid connector #1855
Comments
We are interested in this work , let us know when the connector APIs are updated to expose aggregte pushdown and we can help with this development. |
How about adding druid connector that does not yet support aggregation pushdown? Then we could only focus on aggregation pushdown which itself could be non trivial to do. |
Initial version of Druid connector (#3522) is merged. |
Aggregation pushdown has been added to Trino as of last quarter. Connectors can be updated to take advantage of it. @Parth-Brahmbhatt FYI. |
@antonioromero-pm the PR to handle this in druid has been out #4313 but I just haven't found time to address some of the comments as I moved on to other things. I thought someone from Starburst was going to take over that PR but may be they also haven't found time yet. I will check back once I can carve out some time and try and address the comments so we can actually get this merged in. |
Do we have a plan to support filtering during aggregation pushdown ("filtered aggregator" in Druid) ? |
Also, is JOIN pushdown supported? The Druid lookups may need it |
AbstractTestIntegrationSmokeTest
for Druid (Implement all full AbstractTestIntegrationSmokeTest for Druid #4116)The text was updated successfully, but these errors were encountered: