[GitHub] [drill] cgivre commented on pull request #2387: DRILL-8055: Druid Plugin support for V0.20 and up

[GitBox <gi...@apache.org>]

cgivre commented on pull request #2387:
URL: https://github.com/apache/drill/pull/2387#issuecomment-991801401


   > > @Z0ltrix Thank you very much for this PR! I don't know how much time you have to work on this, but I wanted to suggest that it might be worth adding some additional pushdowns to the Druid plugin. This got a lot easier since DRILL-7985 (#2289) when @vvysotskyi added the `implementor` class. I don't really know what is possible in the new Druid interface, but maybe take a look and see what you think.
   > 
   > Hi @cgivre what do you think about creating a new Ticket for this Change? If everything goes well we could include this PR in the next Release 1.20 so that users are able to use Druid in newer Versions. Regards Christian
   
   I'm fine with that approach. I was a bit concerned about scope creep but I thought I'd mention it since this is the area of the plugin that is affected.   Is this ready for review?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@drill.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org