[GitHub] [incubator-druid] clintropolis commented on issue #8838: Web console: replace (and remove) old consoles

2019-11-06 Thread GitBox
clintropolis commented on issue #8838: Web console: replace (and remove) old consoles URL: https://github.com/apache/incubator-druid/pull/8838#issuecomment-550964787 +1000 (not sure if my review counts since i made one of the commits )

[GitHub] [incubator-druid] vogievetsky opened a new pull request #8838: Web console: replace (and remove) old consoles with new console

2019-11-06 Thread GitBox
vogievetsky opened a new pull request #8838: Web console: replace (and remove) old consoles with new console URL: https://github.com/apache/incubator-druid/pull/8838 Fixes #8150 This PR replaces and removes the legacy coordinator and overlord consoles and replaces them with the new

[GitHub] [incubator-druid] gianm closed pull request #8814: SQL: Add RAND() function.

2019-11-06 Thread GitBox
gianm closed pull request #8814: SQL: Add RAND() function. URL: https://github.com/apache/incubator-druid/pull/8814 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [incubator-druid] gianm commented on issue #8789: SQL RAND function

2019-11-06 Thread GitBox
gianm commented on issue #8789: SQL RAND function URL: https://github.com/apache/incubator-druid/issues/8789#issuecomment-550682645 See https://github.com/apache/incubator-druid/pull/8814#issuecomment-550681737 for why this requires some special care.

[GitHub] [incubator-druid] gianm commented on issue #8814: SQL: Add RAND() function.

2019-11-06 Thread GitBox
gianm commented on issue #8814: SQL: Add RAND() function. URL: https://github.com/apache/incubator-druid/pull/8814#issuecomment-550681737 Actually, I just realized this is a bad idea, for two reasons. 1. The result of the random expression will be potentially cached at query time.

[GitHub] [incubator-druid] jihoonson commented on issue #8837: Kafka indexing task can fail during rolling update

2019-11-06 Thread GitBox
jihoonson commented on issue #8837: Kafka indexing task can fail during rolling update URL: https://github.com/apache/incubator-druid/issues/8837#issuecomment-550595586 This is a release blocker for 0.17. This is an

[GitHub] [incubator-druid] jihoonson opened a new issue #8837: Kafka indexing task can fail during rolling update

2019-11-06 Thread GitBox
jihoonson opened a new issue #8837: Kafka indexing task can fail during rolling update URL: https://github.com/apache/incubator-druid/issues/8837 ### Affected Version When updating 0.15 or earlier -> 0.16 ### Description

[GitHub] [incubator-druid] zhenxiao commented on a change in pull request #8809: Prohibit Futures.addCallback(Future, Callback)

2019-11-06 Thread GitBox
zhenxiao commented on a change in pull request #8809: Prohibit Futures.addCallback(Future, Callback) URL: https://github.com/apache/incubator-druid/pull/8809#discussion_r343408775 ## File path: core/src/main/java/org/apache/druid/java/util/common/concurrent/ListenableFutures.java

[GitHub] [incubator-druid] lgtm-com[bot] commented on issue #8823: Add InputSource and InputFormat interfaces

2019-11-06 Thread GitBox
lgtm-com[bot] commented on issue #8823: Add InputSource and InputFormat interfaces URL: https://github.com/apache/incubator-druid/pull/8823#issuecomment-550553580 This pull request **fixes 1 alert** when merging e942a2139a3f8e44187a1686274db136248e5dff into

[GitHub] [incubator-druid] SEKIRO-J edited a comment on issue #8698: add explodeSpec

2019-11-06 Thread GitBox
SEKIRO-J edited a comment on issue #8698: add explodeSpec URL: https://github.com/apache/incubator-druid/pull/8698#issuecomment-550548889 @clintropolis @vogievetsky Finished the support for explodeSpec for Json Parser, with unit test implemented. I think we can go ahead review it

[GitHub] [incubator-druid] SEKIRO-J edited a comment on issue #8698: add explodeSpec

2019-11-06 Thread GitBox
SEKIRO-J edited a comment on issue #8698: add explodeSpec URL: https://github.com/apache/incubator-druid/pull/8698#issuecomment-550548889 @clintropolis @vogievetsky Finished the support for explodeSpec for Json Parser, with unit test implemented. I think we can go ahead review it

[GitHub] [incubator-druid] SEKIRO-J commented on issue #8698: add explodeSpec

2019-11-06 Thread GitBox
SEKIRO-J commented on issue #8698: add explodeSpec URL: https://github.com/apache/incubator-druid/pull/8698#issuecomment-550548889 @clintropolis @vogievetsky Finished the support for explodeSpec for Json Parser, with unit test implemented. I think we can go ahead and test it in a

[GitHub] [incubator-druid] vogievetsky commented on issue #8790: Flatten Json Field inside TSV?

2019-11-06 Thread GitBox
vogievetsky commented on issue #8790: Flatten Json Field inside TSV? URL: https://github.com/apache/incubator-druid/issues/8790#issuecomment-550547561 This is not currently possible, but would be a cool feature. This is an

[GitHub] [incubator-druid] jihoonson commented on issue #8810: Make submit task API similar to supervisor API

2019-11-06 Thread GitBox
jihoonson commented on issue #8810: Make submit task API similar to supervisor API URL: https://github.com/apache/incubator-druid/pull/8810#issuecomment-550547371 Please fix LGTM alert. This is an automated message from the

[GitHub] [incubator-druid] vogievetsky commented on issue #8818: Eclipse Java Compiler issues fixed

2019-11-06 Thread GitBox
vogievetsky commented on issue #8818: Eclipse Java Compiler issues fixed URL: https://github.com/apache/incubator-druid/pull/8818#issuecomment-550547126 I think this needs to merge in master to pass TeamCity This is an

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8748: Use earliest offset on kafka newly discovered partitions

2019-11-06 Thread GitBox
ccaominh commented on a change in pull request #8748: Use earliest offset on kafka newly discovered partitions URL: https://github.com/apache/incubator-druid/pull/8748#discussion_r343373804 ## File path:

[GitHub] [incubator-druid] jihoonson merged pull request #8821: Upgrade joda-time to 2.10.5

2019-11-06 Thread GitBox
jihoonson merged pull request #8821: Upgrade joda-time to 2.10.5 URL: https://github.com/apache/incubator-druid/pull/8821 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[incubator-druid] branch master updated (ce3c0da -> 517c146)

2019-11-06 Thread jihoonson
This is an automated email from the ASF dual-hosted git repository. jihoonson pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. from ce3c0da Add note on JDBC libs for lookups (#8825) add 517c146 Upgrade joda-time to 2.10.5 (#8821)

[GitHub] [incubator-druid] jihoonson commented on issue #8821: Upgrade joda-time to 2.10.5

2019-11-06 Thread GitBox
jihoonson commented on issue #8821: Upgrade joda-time to 2.10.5 URL: https://github.com/apache/incubator-druid/pull/8821#issuecomment-550530533 @a2l007 thanks! This is an automated message from the Apache Git Service. To

[GitHub] [incubator-druid] jihoonson commented on issue #8836: Parallel indexing task fails during the rolling update

2019-11-06 Thread GitBox
jihoonson commented on issue #8836: Parallel indexing task fails during the rolling update URL: https://github.com/apache/incubator-druid/issues/8836#issuecomment-550529612 This is a release blocker since it breaks the rolling update.

[GitHub] [incubator-druid] jihoonson opened a new issue #8836: Parallel indexing task fails during the rolling update

2019-11-06 Thread GitBox
jihoonson opened a new issue #8836: Parallel indexing task fails during the rolling update URL: https://github.com/apache/incubator-druid/issues/8836 ### Affected Version 0.16 ### Description There are roughly two types of rolling update. One is stopping MMs

[GitHub] [incubator-druid] SEKIRO-J commented on a change in pull request #8748: Use earliest offset on kafka newly discovered partitions

2019-11-06 Thread GitBox
SEKIRO-J commented on a change in pull request #8748: Use earliest offset on kafka newly discovered partitions URL: https://github.com/apache/incubator-druid/pull/8748#discussion_r343357969 ## File path:

[GitHub] [incubator-druid] gianm commented on a change in pull request #8744: support for array expressions in TransformSpec with ExpressionTransform

2019-11-06 Thread GitBox
gianm commented on a change in pull request #8744: support for array expressions in TransformSpec with ExpressionTransform URL: https://github.com/apache/incubator-druid/pull/8744#discussion_r343355536 ## File path:

[GitHub] [incubator-druid] gianm commented on a change in pull request #8744: support for array expressions in TransformSpec with ExpressionTransform

2019-11-06 Thread GitBox
gianm commented on a change in pull request #8744: support for array expressions in TransformSpec with ExpressionTransform URL: https://github.com/apache/incubator-druid/pull/8744#discussion_r343346060 ## File path: core/src/main/java/org/apache/druid/data/input/impl/ParseSpec.java

[GitHub] [incubator-druid] gianm commented on a change in pull request #8744: support for array expressions in TransformSpec with ExpressionTransform

2019-11-06 Thread GitBox
gianm commented on a change in pull request #8744: support for array expressions in TransformSpec with ExpressionTransform URL: https://github.com/apache/incubator-druid/pull/8744#discussion_r343354749 ## File path:

[GitHub] [incubator-druid] suneet-amp commented on a change in pull request #8815: SQL: EARLIEST, LATEST aggregators.

2019-11-06 Thread GitBox
suneet-amp commented on a change in pull request #8815: SQL: EARLIEST, LATEST aggregators. URL: https://github.com/apache/incubator-druid/pull/8815#discussion_r343347084 ## File path:

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows)

2019-11-06 Thread GitBox
clintropolis commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows) URL: https://github.com/apache/incubator-druid/pull/8822#discussion_r343339776 ## File path:

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows)

2019-11-06 Thread GitBox
clintropolis commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows) URL: https://github.com/apache/incubator-druid/pull/8822#discussion_r343338943 ## File path:

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows)

2019-11-06 Thread GitBox
jihoonson commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows) URL: https://github.com/apache/incubator-druid/pull/8822#discussion_r343302979 ## File path:

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows)

2019-11-06 Thread GitBox
jihoonson commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows) URL: https://github.com/apache/incubator-druid/pull/8822#discussion_r343302188 ## File path:

[GitHub] [incubator-druid] gianm commented on issue #8825: Add note on JDBC libs for lookups

2019-11-06 Thread GitBox
gianm commented on issue #8825: Add note on JDBC libs for lookups URL: https://github.com/apache/incubator-druid/pull/8825#issuecomment-550509322 TC is failing, but it's because of the thrift jar issue. I won't bother merging master into this branch, since this change is docs-only.

[GitHub] [incubator-druid] gianm merged pull request #8825: Add note on JDBC libs for lookups

2019-11-06 Thread GitBox
gianm merged pull request #8825: Add note on JDBC libs for lookups URL: https://github.com/apache/incubator-druid/pull/8825 This is an automated message from the Apache Git Service. To respond to the message, please log on

[incubator-druid] branch master updated (5adc821 -> ce3c0da)

2019-11-06 Thread gian
This is an automated email from the ASF dual-hosted git repository. gian pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. from 5adc821 add documentation for druid docker and k8s operator (#8802) add ce3c0da Add note on JDBC libs

[GitHub] [incubator-druid] gianm commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows)

2019-11-06 Thread GitBox
gianm commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows) URL: https://github.com/apache/incubator-druid/pull/8822#discussion_r343318463 ## File path:

[GitHub] [incubator-druid] gianm commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows)

2019-11-06 Thread GitBox
gianm commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows) URL: https://github.com/apache/incubator-druid/pull/8822#discussion_r343324137 ## File path:

[GitHub] [incubator-druid] gianm commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows)

2019-11-06 Thread GitBox
gianm commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows) URL: https://github.com/apache/incubator-druid/pull/8822#discussion_r343314551 ## File path:

[GitHub] [incubator-druid] gianm commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows)

2019-11-06 Thread GitBox
gianm commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows) URL: https://github.com/apache/incubator-druid/pull/8822#discussion_r343317287 ## File path:

[GitHub] [incubator-druid] gianm commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows)

2019-11-06 Thread GitBox
gianm commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows) URL: https://github.com/apache/incubator-druid/pull/8822#discussion_r343316892 ## File path:

[GitHub] [incubator-druid] gianm commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows)

2019-11-06 Thread GitBox
gianm commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows) URL: https://github.com/apache/incubator-druid/pull/8822#discussion_r343317625 ## File path:

[GitHub] [incubator-druid] gianm commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows)

2019-11-06 Thread GitBox
gianm commented on a change in pull request #8822: optimize numeric column null value checking for low filter selectivity (more rows) URL: https://github.com/apache/incubator-druid/pull/8822#discussion_r343319243 ## File path:

[GitHub] [incubator-druid] himanshug merged pull request #8802: add documentation for druid docker and k8s operator

2019-11-06 Thread GitBox
himanshug merged pull request #8802: add documentation for druid docker and k8s operator URL: https://github.com/apache/incubator-druid/pull/8802 This is an automated message from the Apache Git Service. To respond to the

[incubator-druid] branch master updated (5c0fc0a -> 5adc821)

2019-11-06 Thread himanshug
This is an automated email from the ASF dual-hosted git repository. himanshug pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. from 5c0fc0a Fix ambiguity about IndexerSQLMetadataStorageCoordinator.getUsedSegmentsForInterval()

[GitHub] [incubator-druid] stale[bot] commented on issue #6910: Bring GCP functionality to core functionality

2019-11-06 Thread GitBox
stale[bot] commented on issue #6910: Bring GCP functionality to core functionality URL: https://github.com/apache/incubator-druid/issues/6910#issuecomment-550495892 This issue has been marked as stale due to 280 days of inactivity. It will be closed in 4 weeks if no further activity

[GitHub] [incubator-druid] mlubavin-vg edited a comment on issue #8835: Ingestion task fails with NullPointerException during BUILD_SEGMENTS phase

2019-11-06 Thread GitBox
mlubavin-vg edited a comment on issue #8835: Ingestion task fails with NullPointerException during BUILD_SEGMENTS phase URL: https://github.com/apache/incubator-druid/issues/8835#issuecomment-550495167 Looking at git, I see that the FileUtils.copyLarge method that has the NPE inside it

[GitHub] [incubator-druid] mlubavin-vg commented on issue #8835: Ingestion task fails with NullPointerException during BUILD_SEGMENTS phase

2019-11-06 Thread GitBox
mlubavin-vg commented on issue #8835: Ingestion task fails with NullPointerException during BUILD_SEGMENTS phase URL: https://github.com/apache/incubator-druid/issues/8835#issuecomment-550495167 Looking at git, I see that the FileUtils.copyLarge was added in August 2019. And this is the

[GitHub] [incubator-druid] zhenxiao commented on issue #8808: use copy-on-write list in InMemoryAppender

2019-11-06 Thread GitBox
zhenxiao commented on issue #8808: use copy-on-write list in InMemoryAppender URL: https://github.com/apache/incubator-druid/pull/8808#issuecomment-550494309 master merged into this PR. failure seems not related This is an

[GitHub] [incubator-druid] zhenxiao commented on issue #8700: In DirectDruidClient, don't run Future cancellation listener in HTTP library executor

2019-11-06 Thread GitBox
zhenxiao commented on issue #8700: In DirectDruidClient, don't run Future cancellation listener in HTTP library executor URL: https://github.com/apache/incubator-druid/pull/8700#issuecomment-550493860 master merged into this PR. Get failure, seems not related to this change: [ERROR]

[GitHub] [incubator-druid] lgtm-com[bot] commented on issue #8823: Add InputSource and InputFormat interfaces

2019-11-06 Thread GitBox
lgtm-com[bot] commented on issue #8823: Add InputSource and InputFormat interfaces URL: https://github.com/apache/incubator-druid/pull/8823#issuecomment-550490398 This pull request **fixes 1 alert** when merging b7c8b875f1ff9562960c0890137614d0a88c746e into

[GitHub] [incubator-druid] clintropolis edited a comment on issue #8822: optimize numeric column null value checking for low filter selectivity (more rows)

2019-11-06 Thread GitBox
clintropolis edited a comment on issue #8822: optimize numeric column null value checking for low filter selectivity (more rows) URL: https://github.com/apache/incubator-druid/pull/8822#issuecomment-550487638 I slightly adjusted the code to try to cut out a few instructions per loop,

[GitHub] [incubator-druid] mlubavin-vg commented on issue #8835: Ingestion task fails with NullPointerException during BUILD_SEGMENTS phase

2019-11-06 Thread GitBox
mlubavin-vg commented on issue #8835: Ingestion task fails with NullPointerException during BUILD_SEGMENTS phase URL: https://github.com/apache/incubator-druid/issues/8835#issuecomment-550487827 Here is a log line output for the attempt to fetch of the file that fails:

[GitHub] [incubator-druid] clintropolis commented on issue #8822: optimize numeric column null value checking for low filter selectivity (more rows)

2019-11-06 Thread GitBox
clintropolis commented on issue #8822: optimize numeric column null value checking for low filter selectivity (more rows) URL: https://github.com/apache/incubator-druid/pull/8822#issuecomment-550487638 I slightly adjusted the code to try to cut out a few instructions per loop, re-running

[GitHub] [incubator-druid] mlubavin-vg opened a new issue #8835: Ingestion task fails with NullPointerException during BUILD_SEGMENTS phase

2019-11-06 Thread GitBox
mlubavin-vg opened a new issue #8835: Ingestion task fails with NullPointerException during BUILD_SEGMENTS phase URL: https://github.com/apache/incubator-druid/issues/8835 ### Affected Version 0.16.0-incubating I am fairly sure this did not happen with 0.12.3 (we are currently

[GitHub] [incubator-druid] sixtus commented on issue #8830: queryGranularity is hard coded in compaction task

2019-11-06 Thread GitBox
sixtus commented on issue #8830: queryGranularity is hard coded in compaction task URL: https://github.com/apache/incubator-druid/issues/8830#issuecomment-550472343 I think you mean me. I will test this asap. Will likely need documentation too for PR against apache repo.

[GitHub] [incubator-druid] gianm commented on a change in pull request #8814: SQL: Add RAND() function.

2019-11-06 Thread GitBox
gianm commented on a change in pull request #8814: SQL: Add RAND() function. URL: https://github.com/apache/incubator-druid/pull/8814#discussion_r343292920 ## File path: processing/src/main/java/org/apache/druid/query/expression/RandExprMacro.java ## @@ -0,0 +1,95 @@ +/*

[GitHub] [incubator-druid] gianm commented on a change in pull request #8815: SQL: EARLIEST, LATEST aggregators.

2019-11-06 Thread GitBox
gianm commented on a change in pull request #8815: SQL: EARLIEST, LATEST aggregators. URL: https://github.com/apache/incubator-druid/pull/8815#discussion_r343290433 ## File path: sql/src/test/java/org/apache/druid/sql/calcite/CalciteQueryTest.java ## @@ -889,6 +896,123 @@

[GitHub] [incubator-druid] gianm commented on a change in pull request #8815: SQL: EARLIEST, LATEST aggregators.

2019-11-06 Thread GitBox
gianm commented on a change in pull request #8815: SQL: EARLIEST, LATEST aggregators. URL: https://github.com/apache/incubator-druid/pull/8815#discussion_r343290083 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/aggregation/builtin/EarliestLatestSqlAggregator.java

[GitHub] [incubator-druid] lgtm-com[bot] commented on issue #8697: HRTR: make pending task execution handling to go through all tasks on not finding worker slots

2019-11-06 Thread GitBox
lgtm-com[bot] commented on issue #8697: HRTR: make pending task execution handling to go through all tasks on not finding worker slots URL: https://github.com/apache/incubator-druid/pull/8697#issuecomment-550468750 This pull request **introduces 1 alert** when merging

[GitHub] [incubator-druid] jihoonson commented on issue #8821: Upgrade joda-time to 2.10.5

2019-11-06 Thread GitBox
jihoonson commented on issue #8821: Upgrade joda-time to 2.10.5 URL: https://github.com/apache/incubator-druid/pull/8821#issuecomment-550463582 Would you please merge master so that LGTM and TC become happy? This is an

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #8809: Prohibit Futures.addCallback(Future, Callback)

2019-11-06 Thread GitBox
jihoonson commented on a change in pull request #8809: Prohibit Futures.addCallback(Future, Callback) URL: https://github.com/apache/incubator-druid/pull/8809#discussion_r343282304 ## File path:

[GitHub] [incubator-druid] gianm commented on issue #8815: SQL: EARLIEST, LATEST aggregators.

2019-11-06 Thread GitBox
gianm commented on issue #8815: SQL: EARLIEST, LATEST aggregators. URL: https://github.com/apache/incubator-druid/pull/8815#issuecomment-550460582 This is blocked on #8834. The null handling fixes there are required for the tests to pass.

[GitHub] [incubator-druid] vogievetsky commented on issue #8834: Fixes, adjustments to numeric null handling and string first/last aggregators.

2019-11-06 Thread GitBox
vogievetsky commented on issue #8834: Fixes, adjustments to numeric null handling and string first/last aggregators. URL: https://github.com/apache/incubator-druid/pull/8834#issuecomment-550458589 Wow, that is a lot of changes. You basically rewrote them. 

[GitHub] [incubator-druid] jihoonson commented on issue #8830: queryGranularity is hard coded in compaction task

2019-11-06 Thread GitBox
jihoonson commented on issue #8830: queryGranularity is hard coded in compaction task URL: https://github.com/apache/incubator-druid/issues/8830#issuecomment-550457590 Hi @sirpkt, thanks for your report. This is just not implemented yet. I think basically the change in

[GitHub] [incubator-druid] jihoonson merged pull request #8564: Fix ambiguity about IndexerSQLMetadataStorageCoordinator.getUsedSegmentsForInterval() returning only non-overshadowed or all used segmen

2019-11-06 Thread GitBox
jihoonson merged pull request #8564: Fix ambiguity about IndexerSQLMetadataStorageCoordinator.getUsedSegmentsForInterval() returning only non-overshadowed or all used segments URL: https://github.com/apache/incubator-druid/pull/8564

[incubator-druid] branch master updated (7b77cf1 -> 5c0fc0a)

2019-11-06 Thread jihoonson
This is an automated email from the ASF dual-hosted git repository. jihoonson pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. from 7b77cf1 Update README.md (#8829) add 5c0fc0a Fix ambiguity about

[GitHub] [incubator-druid] gianm opened a new pull request #8834: Fixes, adjustments to numeric null handling and string first/last aggregators.

2019-11-06 Thread GitBox
gianm opened a new pull request #8834: Fixes, adjustments to numeric null handling and string first/last aggregators. URL: https://github.com/apache/incubator-druid/pull/8834 There is a class of bugs due to the fact that BaseObjectColumnValueSelector has both "getObject" and "isNull"

[incubator-druid-website] branch asf-site updated (a6246ea -> 27d5345)

2019-11-06 Thread vogievetsky
This is an automated email from the ASF dual-hosted git repository. vogievetsky pushed a change to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-druid-website.git. from a6246ea Merge pull request #45 from implydata/autobuild new fea5aeb Autobuild

[GitHub] [incubator-druid-website] vogievetsky merged pull request #46: Autobuild

2019-11-06 Thread GitBox
vogievetsky merged pull request #46: Autobuild URL: https://github.com/apache/incubator-druid-website/pull/46 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [incubator-druid-website] vogievetsky opened a new pull request #46: Autobuild

2019-11-06 Thread GitBox
vogievetsky opened a new pull request #46: Autobuild URL: https://github.com/apache/incubator-druid-website/pull/46 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8748: Use earliest offset on kafka newly discovered partitions

2019-11-06 Thread GitBox
ccaominh commented on a change in pull request #8748: Use earliest offset on kafka newly discovered partitions URL: https://github.com/apache/incubator-druid/pull/8748#discussion_r343244413 ## File path:

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8748: Use earliest offset on kafka newly discovered partitions

2019-11-06 Thread GitBox
ccaominh commented on a change in pull request #8748: Use earliest offset on kafka newly discovered partitions URL: https://github.com/apache/incubator-druid/pull/8748#discussion_r343241961 ## File path:

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8748: Use earliest offset on kafka newly discovered partitions

2019-11-06 Thread GitBox
ccaominh commented on a change in pull request #8748: Use earliest offset on kafka newly discovered partitions URL: https://github.com/apache/incubator-druid/pull/8748#discussion_r343254505 ## File path:

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8748: Use earliest offset on kafka newly discovered partitions

2019-11-06 Thread GitBox
ccaominh commented on a change in pull request #8748: Use earliest offset on kafka newly discovered partitions URL: https://github.com/apache/incubator-druid/pull/8748#discussion_r343254125 ## File path:

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8748: Use earliest offset on kafka newly discovered partitions

2019-11-06 Thread GitBox
ccaominh commented on a change in pull request #8748: Use earliest offset on kafka newly discovered partitions URL: https://github.com/apache/incubator-druid/pull/8748#discussion_r343249302 ## File path:

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8748: Use earliest offset on kafka newly discovered partitions

2019-11-06 Thread GitBox
ccaominh commented on a change in pull request #8748: Use earliest offset on kafka newly discovered partitions URL: https://github.com/apache/incubator-druid/pull/8748#discussion_r343243986 ## File path:

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8748: Use earliest offset on kafka newly discovered partitions

2019-11-06 Thread GitBox
ccaominh commented on a change in pull request #8748: Use earliest offset on kafka newly discovered partitions URL: https://github.com/apache/incubator-druid/pull/8748#discussion_r343246473 ## File path:

[GitHub] [incubator-druid-website-src] fjy merged pull request #77: refreshing more content

2019-11-06 Thread GitBox
fjy merged pull request #77: refreshing more content URL: https://github.com/apache/incubator-druid-website-src/pull/77 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[incubator-druid-website-src] 01/01: Merge pull request #77 from apache/o

2019-11-06 Thread fjy
This is an automated email from the ASF dual-hosted git repository. fjy pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid-website-src.git commit 227b0462c4a86052c363f2f0db707373b2dbecd4 Merge: 4240030 6131fce Author: Fangjin Yang AuthorDate: Wed

[incubator-druid-website-src] branch master updated (4240030 -> 227b046)

2019-11-06 Thread fjy
This is an automated email from the ASF dual-hosted git repository. fjy pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid-website-src.git. from 4240030 Merge pull request #76 from apache/b add 8bc07dc refreshing more content add

[incubator-druid-website-src] branch o updated (8bc07dc -> 6131fce)

2019-11-06 Thread fjy
This is an automated email from the ASF dual-hosted git repository. fjy pushed a change to branch o in repository https://gitbox.apache.org/repos/asf/incubator-druid-website-src.git. from 8bc07dc refreshing more content add 6131fce fix No new revisions were added by this update.

[incubator-druid-website-src] 01/01: refreshing more content

2019-11-06 Thread fjy
This is an automated email from the ASF dual-hosted git repository. fjy pushed a commit to branch o in repository https://gitbox.apache.org/repos/asf/incubator-druid-website-src.git commit 8bc07dcfa53a71ceec38d308d9897a6db548c6a9 Author: fjy AuthorDate: Wed Nov 6 09:40:20 2019 -0800

[incubator-druid-website-src] branch o created (now 8bc07dc)

2019-11-06 Thread fjy
This is an automated email from the ASF dual-hosted git repository. fjy pushed a change to branch o in repository https://gitbox.apache.org/repos/asf/incubator-druid-website-src.git. at 8bc07dc refreshing more content This branch includes the following new commits: new 8bc07dc

[incubator-druid] branch master updated (7addfc2 -> 7b77cf1)

2019-11-06 Thread fjy
This is an automated email from the ASF dual-hosted git repository. fjy pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. from 7addfc2 Web console: fine grained capabilities / graceful degradation (#8805) add 7b77cf1 Update

[GitHub] [incubator-druid] fjy merged pull request #8829: Update README.md

2019-11-06 Thread GitBox
fjy merged pull request #8829: Update README.md URL: https://github.com/apache/incubator-druid/pull/8829 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [incubator-druid] leventov commented on issue #8808: use copy-on-write list in InMemoryAppender

2019-11-06 Thread GitBox
leventov commented on issue #8808: use copy-on-write list in InMemoryAppender URL: https://github.com/apache/incubator-druid/pull/8808#issuecomment-550400253 @zhenxiao please merge the master branch into your PR branch to make the CI pass.

[GitHub] [incubator-druid] leventov commented on issue #8700: In DirectDruidClient, don't run Future cancellation listener in HTTP library executor

2019-11-06 Thread GitBox
leventov commented on issue #8700: In DirectDruidClient, don't run Future cancellation listener in HTTP library executor URL: https://github.com/apache/incubator-druid/pull/8700#issuecomment-550399937 @zhenxiao please merge the master branch into your PR branch to make the CI pass.

[GitHub] [incubator-druid] stelios313 opened a new issue #8833: SQL query fails in semi join when using NOT IN with subquery

2019-11-06 Thread GitBox
stelios313 opened a new issue #8833: SQL query fails in semi join when using NOT IN with subquery URL: https://github.com/apache/incubator-druid/issues/8833 ### Affected Version 0.16.0 ### Description Executing an SQL query from UI, ``` SELECT dimension,

[GitHub] [incubator-druid] stale[bot] commented on issue #6965: Feature request: "Feature scaling"

2019-11-06 Thread GitBox
stale[bot] commented on issue #6965: Feature request: "Feature scaling" URL: https://github.com/apache/incubator-druid/issues/6965#issuecomment-550373284 This issue has been marked as stale due to 280 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If this

[GitHub] [incubator-druid] superivaj opened a new pull request #8832: SQL Server Metadata store fix

2019-11-06 Thread GitBox
superivaj opened a new pull request #8832: SQL Server Metadata store fix URL: https://github.com/apache/incubator-druid/pull/8832 ### Description The SQL Server Metadata store throws the following exception when creating a new compaction rule in the coordinator. This PR fixes the cause

[GitHub] [incubator-druid] Jaymin-adzmedia commented on issue #8820: groupBy() query is not working in Node Js

2019-11-06 Thread GitBox
Jaymin-adzmedia commented on issue #8820: groupBy() query is not working in Node Js URL: https://github.com/apache/incubator-druid/issues/8820#issuecomment-550283208 Hello vogievetsky, Here is my query in node: > var queryRep = client.timeseries(); >

[GitHub] [incubator-druid] sixtus commented on issue #8830: queryGranularity is hard coded in compaction task

2019-11-06 Thread GitBox
sixtus commented on issue #8830: queryGranularity is hard coded in compaction task URL: https://github.com/apache/incubator-druid/issues/8830#issuecomment-550283038 Is it as simple as https://github.com/liquidm/druid/pull/5/files ?

[GitHub] [incubator-druid] jessp01 commented on issue #8305: Fix to always use end sequenceNumber for reset

2019-11-06 Thread GitBox
jessp01 commented on issue #8305: Fix to always use end sequenceNumber for reset URL: https://github.com/apache/incubator-druid/pull/8305#issuecomment-550260054 Never mind, upgraded to 0.16.0 already. The problem is solved there since this pull was merged.

[GitHub] [incubator-druid] littlebear-xbz opened a new issue #8831: Enable basic security with error

2019-11-06 Thread GitBox
littlebear-xbz opened a new issue #8831: Enable basic security with error URL: https://github.com/apache/incubator-druid/issues/8831 ### Affected Version durid version: apache-druid-0.16.0-incubating jdk version : jdk1.8.0_172-cloudera ### Description

[GitHub] [incubator-druid] sixtus opened a new issue #8830: queryGranularity is hard coded in compaction task

2019-11-06 Thread GitBox
sixtus opened a new issue #8830: queryGranularity is hard coded in compaction task URL: https://github.com/apache/incubator-druid/issues/8830 https://github.com/apache/incubator-druid/blob/master/indexing-service/src/main/java/org/apache/druid/indexing/common/task/CompactionTask.java#L593

[GitHub] [incubator-druid] lgtm-com[bot] commented on issue #8810: Make submit task API similar to supervisor API

2019-11-06 Thread GitBox
lgtm-com[bot] commented on issue #8810: Make submit task API similar to supervisor API URL: https://github.com/apache/incubator-druid/pull/8810#issuecomment-550192675 This pull request **introduces 1 alert** when merging 8d0c0a675e19f1f028861a186a5c267d3c653eda into