[GitHub] jihoonson commented on issue #6173: Composite request logger doesn't invoke @LifeCycleStart and @LifeCycleStop methods on its dependencies

2018-08-14 Thread GitBox
jihoonson commented on issue #6173: Composite request logger doesn't invoke @LifeCycleStart and @LifeCycleStop methods on its dependencies URL: https://github.com/apache/incubator-druid/pull/6173#issuecomment-413089187 @gianm sounds good to me.

[GitHub] jihoonson commented on issue #6155: Fix three bugs with segment publishing.

2018-08-14 Thread GitBox
jihoonson commented on issue #6155: Fix three bugs with segment publishing. URL: https://github.com/apache/incubator-druid/pull/6155#issuecomment-413089530 Hmm, now some unit tests are failing and looks legitimate. ``` Failed tests:

[GitHub] kaijianding opened a new pull request #6178: trigger clean dir when unzip fails with IOException

2018-08-14 Thread GitBox
kaijianding opened a new pull request #6178: trigger clean dir when unzip fails with IOException URL: https://github.com/apache/incubator-druid/pull/6178 SegmentManager, SegmentLoaderLocalCacheManager, and many pullers catch IOException and wrap IOException as SegmentLoadingException, and

[GitHub] samarthjain commented on issue #6173: Composite request logger doesn't invoke @LifeCycleStart and @LifeCycleStop methods on its dependencies

2018-08-15 Thread GitBox
samarthjain commented on issue #6173: Composite request logger doesn't invoke @LifeCycleStart and @LifeCycleStop methods on its dependencies URL: https://github.com/apache/incubator-druid/pull/6173#issuecomment-413104150 Thanks for the reviews. I have updated the PR.

[GitHub] clintropolis commented on a change in pull request #6016: Druid 'Shapeshifting' Columns

2018-08-15 Thread GitBox
clintropolis commented on a change in pull request #6016: Druid 'Shapeshifting' Columns URL: https://github.com/apache/incubator-druid/pull/6016#discussion_r210181585 ## File path: processing/src/main/java/io/druid/segment/data/codecs/ints/RunLengthBytePackedIntFormEncoder.java

[GitHub] sanastas commented on issue #5698: Oak: New Concurrent Key-Value Map

2018-08-16 Thread GitBox
sanastas commented on issue #5698: Oak: New Concurrent Key-Value Map URL: https://github.com/apache/incubator-druid/issues/5698#issuecomment-413513916 Here is the latest pull request #6184 : https://github.com/apache/incubator-druid/pull/6184

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r209743197 ## File path: docs/content/querying/sql.md ## @@ -481,6 +485,77 @@ SELECT * FROM

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r209744738 ## File path: docs/content/querying/sql.md ## @@ -481,6 +485,77 @@ SELECT * FROM

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r210663939 ## File path: sql/src/main/java/io/druid/sql/calcite/schema/SystemSchema.java ## @@ -0,0 +1,435 @@ +/* +

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r209748999 ## File path: sql/src/main/java/io/druid/sql/calcite/schema/SystemSchema.java ## @@ -0,0 +1,435 @@ +/* +

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r208006429 ## File path: docs/content/querying/sql.md ## @@ -481,6 +485,77 @@ SELECT * FROM

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r210627821 ## File path: sql/src/main/java/io/druid/sql/calcite/schema/SystemSchema.java ## @@ -0,0 +1,435 @@ +/* +

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r210661609 ## File path: sql/src/main/java/io/druid/sql/calcite/schema/SystemSchema.java ## @@ -0,0 +1,435 @@ +/* +

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r210663473 ## File path: sql/src/main/java/io/druid/sql/calcite/schema/SystemSchema.java ## @@ -0,0 +1,435 @@ +/* +

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r207957383 ## File path: docs/content/querying/sql.md ## @@ -481,6 +485,77 @@ SELECT * FROM

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r210677565 ## File path: sql/src/main/java/io/druid/sql/calcite/schema/SystemSchema.java ## @@ -0,0 +1,435 @@ +/* +

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r210660722 ## File path: sql/src/main/java/io/druid/sql/calcite/schema/SystemSchema.java ## @@ -0,0 +1,435 @@ +/* +

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r210677643 ## File path: sql/src/main/java/io/druid/sql/calcite/schema/SystemSchema.java ## @@ -0,0 +1,435 @@ +/* +

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r210662196 ## File path: sql/src/main/java/io/druid/sql/calcite/schema/SystemSchema.java ## @@ -0,0 +1,435 @@ +/* +

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r210662719 ## File path: sql/src/main/java/io/druid/sql/calcite/schema/SystemSchema.java ## @@ -0,0 +1,435 @@ +/* +

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r210659443 ## File path: sql/src/main/java/io/druid/sql/calcite/schema/SystemSchema.java ## @@ -0,0 +1,435 @@ +/* +

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r208004645 ## File path: docs/content/querying/sql.md ## @@ -481,6 +485,77 @@ SELECT * FROM

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r207922200 ## File path: docs/content/querying/sql.md ## @@ -481,6 +485,77 @@ SELECT * FROM

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r210669152 ## File path: sql/src/main/java/io/druid/sql/calcite/schema/SystemSchema.java ## @@ -0,0 +1,435 @@ +/* +

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r208005582 ## File path: docs/content/querying/sql.md ## @@ -481,6 +485,77 @@ SELECT * FROM

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r209758183 ## File path: sql/src/main/java/io/druid/sql/calcite/schema/SystemSchema.java ## @@ -0,0 +1,435 @@ +/* +

[GitHub] gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989)

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6094: Introduce SystemSchema tables (#5989) URL: https://github.com/apache/incubator-druid/pull/6094#discussion_r209743270 ## File path: docs/content/querying/sql.md ## @@ -481,6 +485,77 @@ SELECT * FROM

[GitHub] QiuMM commented on issue #6115: Druid should have a 'shutdownAllTasks' API for a dataSource

2018-08-16 Thread GitBox
QiuMM commented on issue #6115: Druid should have a 'shutdownAllTasks' API for a dataSource URL: https://github.com/apache/incubator-druid/issues/6115#issuecomment-413630853 Sounds good, I also want this. I opened a pr to support it.

[GitHub] jihoonson closed pull request #6070: Use JUnit TemporaryFolder rule instead of system temp folder

2018-08-16 Thread GitBox
jihoonson closed pull request #6070: Use JUnit TemporaryFolder rule instead of system temp folder URL: https://github.com/apache/incubator-druid/pull/6070 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] QiuMM opened a new pull request #6185: 'shutdownAllTasks' API for a dataSource

2018-08-16 Thread GitBox
QiuMM opened a new pull request #6185: 'shutdownAllTasks' API for a dataSource URL: https://github.com/apache/incubator-druid/pull/6185 #6115 This is an automated message from the Apache Git Service. To respond to the

[GitHub] gianm commented on a change in pull request #6185: 'shutdownAllTasks' API for a dataSource

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6185: 'shutdownAllTasks' API for a dataSource URL: https://github.com/apache/incubator-druid/pull/6185#discussion_r210687548 ## File path: indexing-service/src/main/java/io/druid/indexing/overlord/http/OverlordResource.java ##

[GitHub] gianm commented on a change in pull request #6185: 'shutdownAllTasks' API for a dataSource

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6185: 'shutdownAllTasks' API for a dataSource URL: https://github.com/apache/incubator-druid/pull/6185#discussion_r210685730 ## File path: docs/content/operations/api-reference.md ## @@ -378,6 +378,9 @@ Endpoint for submitting

[GitHub] jihoonson commented on a change in pull request #6162: Add maven.exec.xxx.skip option for exec-maven-plugin

2018-08-16 Thread GitBox
jihoonson commented on a change in pull request #6162: Add maven.exec.xxx.skip option for exec-maven-plugin URL: https://github.com/apache/incubator-druid/pull/6162#discussion_r210688935 ## File path: docs/content/development/build.md ## @@ -22,7 +22,7 @@ mvn clean

[GitHub] jihoonson commented on a change in pull request #6162: Add maven.exec.xxx.skip option for exec-maven-plugin

2018-08-16 Thread GitBox
jihoonson commented on a change in pull request #6162: Add maven.exec.xxx.skip option for exec-maven-plugin URL: https://github.com/apache/incubator-druid/pull/6162#discussion_r210687869 ## File path: distribution/pom.xml ## @@ -174,6 +83,111 @@ +

[GitHub] gianm commented on a change in pull request #6185: 'shutdownAllTasks' API for a dataSource

2018-08-16 Thread GitBox
gianm commented on a change in pull request #6185: 'shutdownAllTasks' API for a dataSource URL: https://github.com/apache/incubator-druid/pull/6185#discussion_r210697305 ## File path: indexing-service/src/main/java/io/druid/indexing/overlord/http/OverlordResource.java ##

[GitHub] jihoonson commented on a change in pull request #6186: Serde test for JdbcExtractionNamespace.

2018-08-16 Thread GitBox
jihoonson commented on a change in pull request #6186: Serde test for JdbcExtractionNamespace. URL: https://github.com/apache/incubator-druid/pull/6186#discussion_r210723854 ## File path: common/src/main/java/io/druid/metadata/MetadataStorageConnectorConfig.java ## @@

[GitHub] QiuMM commented on a change in pull request #6185: 'shutdownAllTasks' API for a dataSource

2018-08-16 Thread GitBox
QiuMM commented on a change in pull request #6185: 'shutdownAllTasks' API for a dataSource URL: https://github.com/apache/incubator-druid/pull/6185#discussion_r210693684 ## File path: indexing-service/src/main/java/io/druid/indexing/overlord/http/OverlordResource.java ##

[GitHub] QiuMM commented on a change in pull request #6185: 'shutdownAllTasks' API for a dataSource

2018-08-16 Thread GitBox
QiuMM commented on a change in pull request #6185: 'shutdownAllTasks' API for a dataSource URL: https://github.com/apache/incubator-druid/pull/6185#discussion_r210699120 ## File path: indexing-service/src/main/java/io/druid/indexing/overlord/http/OverlordResource.java ##

[GitHub] asdf2014 commented on a change in pull request #6162: Add maven.exec.xxx.skip option for exec-maven-plugin

2018-08-16 Thread GitBox
asdf2014 commented on a change in pull request #6162: Add maven.exec.xxx.skip option for exec-maven-plugin URL: https://github.com/apache/incubator-druid/pull/6162#discussion_r210788802 ## File path: docs/content/development/build.md ## @@ -22,7 +22,7 @@ mvn clean package

[GitHub] asdf2014 commented on a change in pull request #6162: Add maven.exec.xxx.skip option for exec-maven-plugin

2018-08-16 Thread GitBox
asdf2014 commented on a change in pull request #6162: Add maven.exec.xxx.skip option for exec-maven-plugin URL: https://github.com/apache/incubator-druid/pull/6162#discussion_r210787456 ## File path: distribution/pom.xml ## @@ -174,6 +83,111 @@ +

[GitHub] asdf2014 commented on a change in pull request #6162: Add maven.exec.xxx.skip option for exec-maven-plugin

2018-08-16 Thread GitBox
asdf2014 commented on a change in pull request #6162: Add maven.exec.xxx.skip option for exec-maven-plugin URL: https://github.com/apache/incubator-druid/pull/6162#discussion_r210788731 ## File path: distribution/pom.xml ## @@ -174,6 +83,111 @@ +

[GitHub] QiuMM commented on a change in pull request #6185: 'shutdownAllTasks' API for a dataSource

2018-08-16 Thread GitBox
QiuMM commented on a change in pull request #6185: 'shutdownAllTasks' API for a dataSource URL: https://github.com/apache/incubator-druid/pull/6185#discussion_r210794916 ## File path: indexing-service/src/main/java/io/druid/indexing/overlord/http/OverlordResource.java ##

[GitHub] asdf2014 commented on a change in pull request #6185: 'shutdownAllTasks' API for a dataSource

2018-08-16 Thread GitBox
asdf2014 commented on a change in pull request #6185: 'shutdownAllTasks' API for a dataSource URL: https://github.com/apache/incubator-druid/pull/6185#discussion_r210798597 ## File path: indexing-service/src/main/java/io/druid/indexing/overlord/http/OverlordResource.java

[GitHub] QiuMM commented on a change in pull request #6185: 'shutdownAllTasks' API for a dataSource

2018-08-17 Thread GitBox
QiuMM commented on a change in pull request #6185: 'shutdownAllTasks' API for a dataSource URL: https://github.com/apache/incubator-druid/pull/6185#discussion_r210820106 ## File path: indexing-service/src/main/java/io/druid/indexing/overlord/http/OverlordResource.java ##

[GitHub] asdf2014 commented on a change in pull request #6185: 'shutdownAllTasks' API for a dataSource

2018-08-17 Thread GitBox
asdf2014 commented on a change in pull request #6185: 'shutdownAllTasks' API for a dataSource URL: https://github.com/apache/incubator-druid/pull/6185#discussion_r210817443 ## File path: indexing-service/src/main/java/io/druid/indexing/overlord/http/OverlordResource.java

[GitHub] asdf2014 commented on a change in pull request #6185: 'shutdownAllTasks' API for a dataSource

2018-08-17 Thread GitBox
asdf2014 commented on a change in pull request #6185: 'shutdownAllTasks' API for a dataSource URL: https://github.com/apache/incubator-druid/pull/6185#discussion_r210836013 ## File path: indexing-service/src/main/java/io/druid/indexing/overlord/http/OverlordResource.java

[GitHub] leventov opened a new pull request #6112: Prohibit LinkedList

2018-08-05 Thread GitBox
leventov opened a new pull request #6112: Prohibit LinkedList URL: https://github.com/apache/incubator-druid/pull/6112 All use cases for LinkedList are covered by ArrayList and ArrayDeque, which are better options. This is

[GitHub] leventov commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges

2018-08-05 Thread GitBox
leventov commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges URL: https://github.com/apache/incubator-druid/pull/5913#discussion_r207755440 ## File path:

[GitHub] leventov commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges

2018-08-05 Thread GitBox
leventov commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges URL: https://github.com/apache/incubator-druid/pull/5913#discussion_r207755064 ## File path:

[GitHub] leventov commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges

2018-08-05 Thread GitBox
leventov commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges URL: https://github.com/apache/incubator-druid/pull/5913#discussion_r207755399 ## File path:

[GitHub] leventov commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges

2018-08-05 Thread GitBox
leventov commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges URL: https://github.com/apache/incubator-druid/pull/5913#discussion_r207754740 ## File path:

[GitHub] gianm opened a new issue #6114: Kafka supervisor with unresolvable bootstrap.servers blocks startup

2018-08-06 Thread GitBox
gianm opened a new issue #6114: Kafka supervisor with unresolvable bootstrap.servers blocks startup URL: https://github.com/apache/incubator-druid/issues/6114 Kafka supervisors with unresolvable bootstrap.servers will block overlord startup. The stack trace looks like this: ```

[GitHub] a2l007 commented on issue #6104: Webhdfs support for orc-extension

2018-08-06 Thread GitBox
a2l007 commented on issue #6104: Webhdfs support for orc-extension URL: https://github.com/apache/incubator-druid/issues/6104#issuecomment-410737949 Yeah ORC does a lot of seeks and the check could be because it may not work well with WebHDFS i guess. I wonder if it would make sense to

[GitHub] a2l007 commented on issue #6104: Webhdfs support for orc-extension

2018-08-07 Thread GitBox
a2l007 commented on issue #6104: Webhdfs support for orc-extension URL: https://github.com/apache/incubator-druid/issues/6104#issuecomment-411105781 Sure I'll investigate if it is indeed a better solution and then work towards contributing it.

[GitHub] gianm commented on issue #6108: more examples needed for druid sql

2018-08-06 Thread GitBox
gianm commented on issue #6108: more examples needed for druid sql URL: https://github.com/apache/incubator-druid/issues/6108#issuecomment-410796733 > @gianm regarding 4. is the behavior of native queries (topn, timeseries, groupby) the same as sql for this case? in that will they omit

[GitHub] jihoonson edited a comment on issue #5492: Native parallel batch indexing without shuffle

2018-08-07 Thread GitBox
jihoonson edited a comment on issue #5492: Native parallel batch indexing without shuffle URL: https://github.com/apache/incubator-druid/pull/5492#issuecomment-410923528 @josephglanville in perfect roll-up, all rows which have the timestamp falling in the same interval should be in the

[GitHub] drcrallen commented on issue #5492: Native parallel batch indexing without shuffle

2018-08-07 Thread GitBox
drcrallen commented on issue #5492: Native parallel batch indexing without shuffle URL: https://github.com/apache/incubator-druid/pull/5492#issuecomment-411124748 Tagged for special callout in release notes This is an

[GitHub] asdf2014 opened a new pull request #6162: Add maven.exec.xxx.skip option for exec-maven-plugin

2018-08-13 Thread GitBox
asdf2014 opened a new pull request #6162: Add maven.exec.xxx.skip option for exec-maven-plugin URL: https://github.com/apache/incubator-druid/pull/6162 Try to fix [#6160](https://github.com/apache/incubator-druid/issues/6160)

[GitHub] leventov opened a new issue #6161: Extension list in docs still mentions druid-caffeine-cache

2018-08-12 Thread GitBox
leventov opened a new issue #6161: Extension list in docs still mentions druid-caffeine-cache URL: https://github.com/apache/incubator-druid/issues/6161 http://druid.io/docs/0.12.2/development/extensions.html still mentions `druid-caffeine-cache`, although it doesn't exist anymore.

[GitHub] wpcnjupt opened a new issue #6163: java.lang.NullPointerException: taskGroupId when middlemanager rolling update to 0.12.2

2018-08-13 Thread GitBox
wpcnjupt opened a new issue #6163: java.lang.NullPointerException: taskGroupId when middlemanager rolling update to 0.12.2 URL: https://github.com/apache/incubator-druid/issues/6163 hi I am rolling update to 0.12.2. I have updated the Historical nodes and one middlemanager node.

[GitHub] asdf2014 opened a new pull request #6158: Make time-related variables more readable

2018-08-12 Thread GitBox
asdf2014 opened a new pull request #6158: Make time-related variables more readable URL: https://github.com/apache/incubator-druid/pull/6158 Make time-related variables more readable This is an automated message from the

[GitHub] pdeva opened a new issue #6164: extensions not updated for 0.12.2

2018-08-13 Thread GitBox
pdeva opened a new issue #6164: extensions not updated for 0.12.2 URL: https://github.com/apache/incubator-druid/issues/6164 example, io.druid.extensions.contrib:druid-redis-cache the latest version remains 0.12.1 This is an

[GitHub] meazza commented on issue #5480: SQL: not in operator can cause NullPointerException

2018-08-13 Thread GitBox
meazza commented on issue #5480: SQL: not in operator can cause NullPointerException URL: https://github.com/apache/incubator-druid/issues/5480#issuecomment-412438737 ``` { "error": "Unknown exception", "errorMessage": "Cannot build plan for query: SELECT COUNT(*) AS TheCount

[GitHub] asdf2014 commented on issue #6162: Add maven.exec.xxx.skip option for exec-maven-plugin

2018-08-13 Thread GitBox
asdf2014 commented on issue #6162: Add maven.exec.xxx.skip option for exec-maven-plugin URL: https://github.com/apache/incubator-druid/pull/6162#issuecomment-412456329 It seems that those maven options are in effect. The travis

[GitHub] QiuMM commented on issue #6154: Ability to specify list of task ports

2018-08-11 Thread GitBox
QiuMM commented on issue #6154: Ability to specify list of task ports URL: https://github.com/apache/incubator-druid/issues/6154#issuecomment-412288677 +1. We have met port conflicts because of running middle manger on Docker containers. If more than two middle manger instance run on a

[GitHub] gianm commented on issue #6152: Fix UriCacheGeneratorTest for Nightly build

2018-08-11 Thread GitBox
gianm commented on issue #6152: Fix UriCacheGeneratorTest for Nightly build URL: https://github.com/apache/incubator-druid/pull/6152#issuecomment-412282563 I think that snowman was meant to make sure that UriExtractionNamespace worked OK when paths have "weird" characters in them. I guess

[GitHub] KenjiTakahashi commented on issue #6154: Ability to specify list of task ports

2018-08-11 Thread GitBox
KenjiTakahashi commented on issue #6154: Ability to specify list of task ports URL: https://github.com/apache/incubator-druid/issues/6154#issuecomment-412286474 +1. We have a use case where Druid and all its' deps are on Linux, in Docker containers. But the ingestion service is on

[GitHub] gianm closed issue #6139: Race in testCheckpointForUnknownTaskGroup() of KafkaSupervisorTest

2018-08-11 Thread GitBox
gianm closed issue #6139: Race in testCheckpointForUnknownTaskGroup() of KafkaSupervisorTest URL: https://github.com/apache/incubator-druid/issues/6139 This is an automated message from the Apache Git Service. To respond to

[GitHub] gianm closed pull request #6153: Fix race in testCheckpointForUnknownTaskGroup() of KafkaSupervisorTest

2018-08-11 Thread GitBox
gianm closed pull request #6153: Fix race in testCheckpointForUnknownTaskGroup() of KafkaSupervisorTest URL: https://github.com/apache/incubator-druid/pull/6153 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] pdeva opened a new issue #6166: MAJOR issue: Unrecoverable error in KIS 0.12.2

2018-08-13 Thread GitBox
pdeva opened a new issue #6166: MAJOR issue: Unrecoverable error in KIS 0.12.2 URL: https://github.com/apache/incubator-druid/issues/6166 all the tasks keep restarting with this error in the logs: I tried: - upgrading node from 0.12.1 to 0.12.2 - spinning a whole new server and

[GitHub] pdeva opened a new issue #6165: KIS stuck in reboot loop druid 0.12.2

2018-08-13 Thread GitBox
pdeva opened a new issue #6165: KIS stuck in reboot loop druid 0.12.2 URL: https://github.com/apache/incubator-druid/issues/6165 details here: https://groups.google.com/forum/#!topic/druid-user/K7YmcBJM6T4 please help!

[GitHub] asdf2014 commented on issue #5955: The annotation handler 'io.druid.annotations.SubclassesMustBePublicAnnotationProcessor' could not be found

2018-08-10 Thread GitBox
asdf2014 commented on issue #5955: The annotation handler 'io.druid.annotations.SubclassesMustBePublicAnnotationProcessor' could not be found URL: https://github.com/apache/incubator-druid/issues/5955#issuecomment-412013410 I confirm that after upgrading Intellij's idea to v2018.2, it no

[GitHub] asdf2014 removed a comment on issue #5955: The annotation handler 'io.druid.annotations.SubclassesMustBePublicAnnotationProcessor' could not be found

2018-08-10 Thread GitBox
asdf2014 removed a comment on issue #5955: The annotation handler 'io.druid.annotations.SubclassesMustBePublicAnnotationProcessor' could not be found URL: https://github.com/apache/incubator-druid/issues/5955#issuecomment-412013410 I confirm that after upgrading Intellij's idea to

[GitHub] RestfulBlue commented on issue #5006: No protections for select query

2018-08-10 Thread GitBox
RestfulBlue commented on issue #5006: No protections for select query URL: https://github.com/apache/incubator-druid/issues/5006#issuecomment-412031674 @asdf2014 we use grafana as main gui, we have our self developed druid datasource and we have good restrictions as well. Using grafana

[GitHub] asdf2014 commented on issue #5849: IntelliJ Annotation processor prevents running tests in IntelliJ

2018-08-10 Thread GitBox
asdf2014 commented on issue #5849: IntelliJ Annotation processor prevents running tests in IntelliJ URL: https://github.com/apache/incubator-druid/issues/5849#issuecomment-412013779 I also confirmed that after upgrading Intellij Idea to v2018.2, it no longer happens. Thanks all.

[GitHub] asdf2014 commented on issue #6090: Fix missing exception handling as part of `io.druid.java.util.http.client.netty.HttpClientPipelineFactory`

2018-08-10 Thread GitBox
asdf2014 commented on issue #6090: Fix missing exception handling as part of `io.druid.java.util.http.client.netty.HttpClientPipelineFactory` URL: https://github.com/apache/incubator-druid/pull/6090#issuecomment-412017812 @jihoonson Thanks a lot, I have seen that travis job has been

[GitHub] leventov commented on a change in pull request #5957: Renamed 'Generic Column' -> 'Numeric Column'; Fixed a few resource leaks in processing; Fixed a bug in SingleStringInputDimensionSelector

2018-08-09 Thread GitBox
leventov commented on a change in pull request #5957: Renamed 'Generic Column' -> 'Numeric Column'; Fixed a few resource leaks in processing; Fixed a bug in SingleStringInputDimensionSelector; misc refinements URL: https://github.com/apache/incubator-druid/pull/5957#discussion_r209110956

[GitHub] niketh commented on issue #6138: [Proposal] Move Task to move segments while migrating deep storages

2018-08-09 Thread GitBox
niketh commented on issue #6138: [Proposal] Move Task to move segments while migrating deep storages URL: https://github.com/apache/incubator-druid/issues/6138#issuecomment-411938358 Agreed. I will take a stab at it. This

[GitHub] jon-wei closed pull request #6141: Fix kafka tutorial typo

2018-08-09 Thread GitBox
jon-wei closed pull request #6141: Fix kafka tutorial typo URL: https://github.com/apache/incubator-druid/pull/6141 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] jon-wei opened a new pull request #6148: [Backport] New quickstart and tutorials

2018-08-09 Thread GitBox
jon-wei opened a new pull request #6148: [Backport] New quickstart and tutorials URL: https://github.com/apache/incubator-druid/pull/6148 Backport of #6126 to 0.12.3 This is an automated message from the Apache Git Service.

[GitHub] jihoonson closed issue #6116: Druid 0.12.2 release notes

2018-08-09 Thread GitBox
jihoonson closed issue #6116: Druid 0.12.2 release notes URL: https://github.com/apache/incubator-druid/issues/6116 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] jon-wei closed pull request #6137: [Backport] Add docs for virtual columns and transform specs

2018-08-09 Thread GitBox
jon-wei closed pull request #6137: [Backport] Add docs for virtual columns and transform specs URL: https://github.com/apache/incubator-druid/pull/6137 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] jon-wei commented on issue #6137: [Backport] Add docs for virtual columns and transform specs

2018-08-09 Thread GitBox
jon-wei commented on issue #6137: [Backport] Add docs for virtual columns and transform specs URL: https://github.com/apache/incubator-druid/pull/6137#issuecomment-411927805 Backporting to 0.12.3 instead This is an automated

[GitHub] jihoonson commented on issue #6138: [Proposal] Move Task to move segments while migrating deep storages

2018-08-09 Thread GitBox
jihoonson commented on issue #6138: [Proposal] Move Task to move segments while migrating deep storages URL: https://github.com/apache/incubator-druid/issues/6138#issuecomment-411935893 We already have the moveTask for this purpose. What's the difference?

[GitHub] jon-wei commented on a change in pull request #6126: New quickstart and tutorials

2018-08-09 Thread GitBox
jon-wei commented on a change in pull request #6126: New quickstart and tutorials URL: https://github.com/apache/incubator-druid/pull/6126#discussion_r209117132 ## File path: docs/content/tutorials/tutorial-batch-hadoop.md ## @@ -0,0 +1,247 @@ +--- +layout: doc_page +---

[GitHub] jon-wei commented on a change in pull request #6126: New quickstart and tutorials

2018-08-09 Thread GitBox
jon-wei commented on a change in pull request #6126: New quickstart and tutorials URL: https://github.com/apache/incubator-druid/pull/6126#discussion_r209117123 ## File path: docs/content/tutorials/tutorial-batch-hadoop.md ## @@ -0,0 +1,247 @@ +--- +layout: doc_page +---

[GitHub] jon-wei commented on a change in pull request #6126: New quickstart and tutorials

2018-08-09 Thread GitBox
jon-wei commented on a change in pull request #6126: New quickstart and tutorials URL: https://github.com/apache/incubator-druid/pull/6126#discussion_r209117206 ## File path: docs/content/tutorials/tutorial-kafka.md ## @@ -37,149 +30,56 @@ Start a Kafka broker by running

[GitHub] jon-wei opened a new pull request #6144: [Backport] Separate hadoop and native batch docs more

2018-08-09 Thread GitBox
jon-wei opened a new pull request #6144: [Backport] Separate hadoop and native batch docs more URL: https://github.com/apache/incubator-druid/pull/6144 Backport of #6120 to 0.12.3 This is an automated message from the Apache

[GitHub] asdf2014 commented on issue #6090: Fix missing exception handling as part of `io.druid.java.util.http.client.netty.HttpClientPipelineFactory`

2018-08-10 Thread GitBox
asdf2014 commented on issue #6090: Fix missing exception handling as part of `io.druid.java.util.http.client.netty.HttpClientPipelineFactory` URL: https://github.com/apache/incubator-druid/pull/6090#issuecomment-411999244 Hi, @jihoonson . It seems that the failure of travis build job is

[GitHub] leventov commented on a change in pull request #5957: Renamed 'Generic Column' -> 'Numeric Column'; Fixed a few resource leaks in processing; Fixed a bug in SingleStringInputDimensionSelector

2018-08-09 Thread GitBox
leventov commented on a change in pull request #5957: Renamed 'Generic Column' -> 'Numeric Column'; Fixed a few resource leaks in processing; Fixed a bug in SingleStringInputDimensionSelector; misc refinements URL: https://github.com/apache/incubator-druid/pull/5957#discussion_r209113135

[GitHub] surekhasaharan commented on a change in pull request #6126: New quickstart and tutorials

2018-08-09 Thread GitBox
surekhasaharan commented on a change in pull request #6126: New quickstart and tutorials URL: https://github.com/apache/incubator-druid/pull/6126#discussion_r209114320 ## File path: docs/content/tutorials/tutorial-update-data.md ## @@ -0,0 +1,150 @@ +--- +layout: doc_page

[GitHub] surekhasaharan commented on a change in pull request #6126: New quickstart and tutorials

2018-08-09 Thread GitBox
surekhasaharan commented on a change in pull request #6126: New quickstart and tutorials URL: https://github.com/apache/incubator-druid/pull/6126#discussion_r209061274 ## File path: docs/content/tutorials/tutorial-batch-hadoop.md ## @@ -0,0 +1,247 @@ +--- +layout:

[GitHub] surekhasaharan commented on a change in pull request #6126: New quickstart and tutorials

2018-08-09 Thread GitBox
surekhasaharan commented on a change in pull request #6126: New quickstart and tutorials URL: https://github.com/apache/incubator-druid/pull/6126#discussion_r209065442 ## File path: docs/content/tutorials/tutorial-batch-hadoop.md ## @@ -0,0 +1,247 @@ +--- +layout:

[GitHub] surekhasaharan commented on a change in pull request #6126: New quickstart and tutorials

2018-08-09 Thread GitBox
surekhasaharan commented on a change in pull request #6126: New quickstart and tutorials URL: https://github.com/apache/incubator-druid/pull/6126#discussion_r209108427 ## File path: docs/content/tutorials/tutorial-kafka.md ## @@ -37,149 +30,56 @@ Start a Kafka broker by

[GitHub] surekhasaharan commented on a change in pull request #6126: New quickstart and tutorials

2018-08-09 Thread GitBox
surekhasaharan commented on a change in pull request #6126: New quickstart and tutorials URL: https://github.com/apache/incubator-druid/pull/6126#discussion_r209071989 ## File path: docs/content/tutorials/tutorial-compaction.md ## @@ -0,0 +1,103 @@ +--- +layout: doc_page

[GitHub] niketh commented on issue #6138: [Proposal] Move Task to move segments while migrating deep storages

2018-08-09 Thread GitBox
niketh commented on issue #6138: [Proposal] Move Task to move segments while migrating deep storages URL: https://github.com/apache/incubator-druid/issues/6138#issuecomment-411936992 @jihoonson I didn't find it in the documentation http://druid.io/docs/latest/ingestion/tasks.html. I

[GitHub] leventov commented on a change in pull request #5957: Renamed 'Generic Column' -> 'Numeric Column'; Fixed a few resource leaks in processing; Fixed a bug in SingleStringInputDimensionSelector

2018-08-09 Thread GitBox
leventov commented on a change in pull request #5957: Renamed 'Generic Column' -> 'Numeric Column'; Fixed a few resource leaks in processing; Fixed a bug in SingleStringInputDimensionSelector; misc refinements URL: https://github.com/apache/incubator-druid/pull/5957#discussion_r209116416

[GitHub] jihoonson commented on issue #6138: [Proposal] Move Task to move segments while migrating deep storages

2018-08-09 Thread GitBox
jihoonson commented on issue #6138: [Proposal] Move Task to move segments while migrating deep storages URL: https://github.com/apache/incubator-druid/issues/6138#issuecomment-411937694 Yeah, we should document and improve it.

[GitHub] leventov opened a new issue #6140: Is it valid that {Long, Float, Double}{First, Last} aggregator factories return the same comparator from getComprator()

2018-08-09 Thread GitBox
leventov opened a new issue #6140: Is it valid that {Long,Float,Double}{First,Last} aggregator factories return the same comparator from getComprator() URL: https://github.com/apache/incubator-druid/issues/6140 Shouldn't they return inverse comparators? @nishantmonu51 @gianm

[GitHub] jon-wei opened a new pull request #6141: Fix kafka tutorial typo

2018-08-09 Thread GitBox
jon-wei opened a new pull request #6141: Fix kafka tutorial typo URL: https://github.com/apache/incubator-druid/pull/6141 FIxes a reference to "Imply directory" in the new kafka ingestion tutorial. This is an automated

<    3   4   5   6   7   8   9   10   11   12   >