[GitHub] [drill] akkapur commented on issue #1888: DRILL-5956: Add Storage Plugin for Apache Druid

2020-03-03 Thread GitBox
akkapur commented on issue #1888: DRILL-5956: Add Storage Plugin for Apache Druid URL: https://github.com/apache/drill/pull/1888#issuecomment-594262755 > Same comment. If Druid isn't running, the tests that depend on it will fail, so wouldn't we want to fail fast and throw an exception

[GitHub] [drill] akkapur commented on issue #1888: DRILL-5956: Add Storage Plugin for Apache Druid

2020-03-01 Thread GitBox
akkapur commented on issue #1888: DRILL-5956: Add Storage Plugin for Apache Druid URL: https://github.com/apache/drill/pull/1888#issuecomment-593131759 > @akkapur > Thanks for submitting this plugin. I started my review and ran into some issues. Firstly the unit tests that do not

[GitHub] [drill] akkapur commented on issue #1888: DRILL-5956: Add Storage Plugin for Apache Druid

2020-02-29 Thread GitBox
akkapur commented on issue #1888: DRILL-5956: Add Storage Plugin for Apache Druid URL: https://github.com/apache/drill/pull/1888#issuecomment-593001615 @cgivre thanks for taking time to review. I will try to resolve this asap.

[GitHub] [drill] akkapur commented on issue #1888: DRILL-5956: Add Storage Plugin for Apache Druid

2020-02-09 Thread GitBox
akkapur commented on issue #1888: DRILL-5956: Add Storage Plugin for Apache Druid URL: https://github.com/apache/drill/pull/1888#issuecomment-583946402 @cgivre I am running into a runtime exception when running the tests in TestDruidQueries.java. `java.lang.NoClassDefFoundError:

[GitHub] [drill] akkapur commented on issue #1888: DRILL-5956: Add Storage Plugin for Apache Druid

2020-02-09 Thread GitBox
akkapur commented on issue #1888: DRILL-5956: Add Storage Plugin for Apache Druid URL: https://github.com/apache/drill/pull/1888#issuecomment-583916800 > @akkapur I just updated the PR to use the latest build of Drill and rebuilt the protobufs. Great!. Thanks for keeping tabs on