[GitHub] [incubator-druid-website-src] sureddy opened a new pull request #32: Update druid-powered.md

2019-08-01 Thread GitBox
sureddy opened a new pull request #32: Update druid-powered.md URL: https://github.com/apache/incubator-druid-website-src/pull/32 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-druid] fjy merged pull request #8223: Add IPv4 SQL functions

2019-08-01 Thread GitBox
fjy merged pull request #8223: Add IPv4 SQL functions URL: https://github.com/apache/incubator-druid/pull/8223 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[incubator-druid] branch master updated: Add IPv4 SQL functions (#8223)

2019-08-01 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.git The following commit(s) were added to refs/heads/master by this push: new 4bd3bad Add IPv4 SQL functions (#8223)

[GitHub] [incubator-druid] licl2014 commented on issue #3736: Remove throttling on drop segments

2019-08-01 Thread GitBox
licl2014 commented on issue #3736: Remove throttling on drop segments URL: https://github.com/apache/incubator-druid/pull/3736#issuecomment-517541106 "@fjy The reason why "some segments could never get deleted" was because we had maxSegmentsToMove to be set more than

[GitHub] [incubator-druid] jihoonson commented on issue #8061: Native parallel batch indexing with shuffle

2019-08-01 Thread GitBox
jihoonson commented on issue #8061: Native parallel batch indexing with shuffle URL: https://github.com/apache/incubator-druid/issues/8061#issuecomment-517516557 I think two big changes are happening in Druid's indexing service recently. One would be the native parallel indexing and

[GitHub] [incubator-druid] gianm commented on a change in pull request #8223: Add IPv4 SQL functions

2019-08-01 Thread GitBox
gianm commented on a change in pull request #8223: Add IPv4 SQL functions URL: https://github.com/apache/incubator-druid/pull/8223#discussion_r309953626 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/planner/DruidOperatorTable.java ## @@ -243,6 +253,8 @@

[GitHub] [incubator-druid] clintropolis merged pull request #8219: Web-Console: stop log from making request after dialog close, implement query manager

2019-08-01 Thread GitBox
clintropolis merged pull request #8219: Web-Console: stop log from making request after dialog close, implement query manager URL: https://github.com/apache/incubator-druid/pull/8219 This is an automated message from the

[incubator-druid] branch master updated: Web-Console: stop log from making request after dialog close, implement query manager (#8219)

2019-08-01 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/master by this push: new 5528dec Web-Console: stop log from

[incubator-druid] branch master updated: Web-Console: add go to task for datasources (#8210)

2019-08-01 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/master by this push: new 02c3a78 Web-Console: add go to task

[GitHub] [incubator-druid] clintropolis merged pull request #8210: Web-Console: add go to task for datasources

2019-08-01 Thread GitBox
clintropolis merged pull request #8210: Web-Console: add go to task for datasources URL: https://github.com/apache/incubator-druid/pull/8210 This is an automated message from the Apache Git Service. To respond to the

[incubator-druid] branch master updated: correct kerberos doc extension load list (#8224)

2019-08-01 Thread jonwei
This is an automated email from the ASF dual-hosted git repository. jonwei pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/master by this push: new 01c8c82 correct kerberos doc

[GitHub] [incubator-druid] jon-wei commented on issue #8224: correct kerberos doc example extension load list

2019-08-01 Thread GitBox
jon-wei commented on issue #8224: correct kerberos doc example extension load list URL: https://github.com/apache/incubator-druid/pull/8224#issuecomment-517499199 Skipping CI for doc only change This is an automated message

[GitHub] [incubator-druid] jon-wei merged pull request #8224: correct kerberos doc example extension load list

2019-08-01 Thread GitBox
jon-wei merged pull request #8224: correct kerberos doc example extension load list URL: https://github.com/apache/incubator-druid/pull/8224 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-druid] clintropolis opened a new pull request #8224: correct kerberos doc example extension load list

2019-08-01 Thread GitBox
clintropolis opened a new pull request #8224: correct kerberos doc example extension load list URL: https://github.com/apache/incubator-druid/pull/8224 This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-druid] vogievetsky commented on issue #8219: Web-Console: stop log from making request after dialog close, implement query manager

2019-08-01 Thread GitBox
vogievetsky commented on issue #8219: Web-Console: stop log from making request after dialog close, implement query manager URL: https://github.com/apache/incubator-druid/pull/8219#issuecomment-517493744 Looks good to me,  could be merged after travis

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8223: Add IPv4 SQL functions

2019-08-01 Thread GitBox
ccaominh commented on a change in pull request #8223: Add IPv4 SQL functions URL: https://github.com/apache/incubator-druid/pull/8223#discussion_r309929387 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/planner/DruidOperatorTable.java ## @@ -243,6 +253,8 @@

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8219: Web-Console: stop log from making request after dialog close, implement query manager

2019-08-01 Thread GitBox
vogievetsky commented on a change in pull request #8219: Web-Console: stop log from making request after dialog close, implement query manager URL: https://github.com/apache/incubator-druid/pull/8219#discussion_r309929551 ## File path:

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8219: Web-Console: stop log from making request after dialog close, implement query manager

2019-08-01 Thread GitBox
vogievetsky commented on a change in pull request #8219: Web-Console: stop log from making request after dialog close, implement query manager URL: https://github.com/apache/incubator-druid/pull/8219#discussion_r309929635 ## File path:

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8223: Add IPv4 SQL functions

2019-08-01 Thread GitBox
ccaominh commented on a change in pull request #8223: Add IPv4 SQL functions URL: https://github.com/apache/incubator-druid/pull/8223#discussion_r309929387 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/planner/DruidOperatorTable.java ## @@ -243,6 +253,8 @@

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8223: Add IPv4 SQL functions

2019-08-01 Thread GitBox
ccaominh commented on a change in pull request #8223: Add IPv4 SQL functions URL: https://github.com/apache/incubator-druid/pull/8223#discussion_r309928543 ## File path: sql/src/test/java/org/apache/druid/sql/calcite/expression/ExpressionTestHelper.java ## @@ -0,0 +1,236

[GitHub] [incubator-druid] gianm commented on a change in pull request #8223: Add IPv4 SQL functions

2019-08-01 Thread GitBox
gianm commented on a change in pull request #8223: Add IPv4 SQL functions URL: https://github.com/apache/incubator-druid/pull/8223#discussion_r309925278 ## File path: sql/src/test/java/org/apache/druid/sql/calcite/expression/ExpressionTestHelper.java ## @@ -0,0 +1,236 @@

[GitHub] [incubator-druid] gianm commented on a change in pull request #8223: Add IPv4 SQL functions

2019-08-01 Thread GitBox
gianm commented on a change in pull request #8223: Add IPv4 SQL functions URL: https://github.com/apache/incubator-druid/pull/8223#discussion_r309924208 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/planner/DruidOperatorTable.java ## @@ -243,6 +253,8 @@

[GitHub] [incubator-druid] vogievetsky commented on issue #8208: Druid SQL: Fetch latest records from streaming ingestion

2019-08-01 Thread GitBox
vogievetsky commented on issue #8208: Druid SQL: Fetch latest records from streaming ingestion URL: https://github.com/apache/incubator-druid/issues/8208#issuecomment-517483429 This is funny we were just talking about a very similar thing in the [Slack

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8100: SQL support for t-digest based sketch aggregators

2019-08-01 Thread GitBox
jon-wei commented on a change in pull request #8100: SQL support for t-digest based sketch aggregators URL: https://github.com/apache/incubator-druid/pull/8100#discussion_r309921378 ## File path:

[GitHub] [incubator-druid] vogievetsky commented on issue #8025: PagingIdentifiers are not right when query data with select query

2019-08-01 Thread GitBox
vogievetsky commented on issue #8025: PagingIdentifiers are not right when query data with select query URL: https://github.com/apache/incubator-druid/issues/8025#issuecomment-517480600 @duanxuelin the scan query streams results which is like a built in pagination

[GitHub] [incubator-druid] samarthjain commented on a change in pull request #8100: SQL support for t-digest based sketch aggregators

2019-08-01 Thread GitBox
samarthjain commented on a change in pull request #8100: SQL support for t-digest based sketch aggregators URL: https://github.com/apache/incubator-druid/pull/8100#discussion_r309920428 ## File path:

[GitHub] [incubator-druid] vogievetsky commented on issue #8210: Web-Console: add go to task for datasources

2019-08-01 Thread GitBox
vogievetsky commented on issue #8210: Web-Console: add go to task for datasources URL: https://github.com/apache/incubator-druid/pull/8210#issuecomment-517475983 Looks great, thank you for addressing my feedback  This

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8100: SQL support for t-digest based sketch aggregators

2019-08-01 Thread GitBox
jon-wei commented on a change in pull request #8100: SQL support for t-digest based sketch aggregators URL: https://github.com/apache/incubator-druid/pull/8100#discussion_r309914605 ## File path:

[GitHub] [incubator-druid] AlexanderSaydakov commented on issue #8095: Incorrect quantiles result produced by quantilesDoublesSketch

2019-08-01 Thread GitBox
AlexanderSaydakov commented on issue #8095: Incorrect quantiles result produced by quantilesDoublesSketch URL: https://github.com/apache/incubator-druid/issues/8095#issuecomment-517472421 In the log above I see NonNativeWritableMemoryImpl that suggests using big endian byte order. I

[GitHub] [incubator-druid] samarthjain commented on a change in pull request #8100: SQL support for t-digest based sketch aggregators

2019-08-01 Thread GitBox
samarthjain commented on a change in pull request #8100: SQL support for t-digest based sketch aggregators URL: https://github.com/apache/incubator-druid/pull/8100#discussion_r309912709 ## File path:

[GitHub] [incubator-druid] ccaominh opened a new pull request #8223: Add IPv4 SQL functions

2019-08-01 Thread GitBox
ccaominh opened a new pull request #8223: Add IPv4 SQL functions URL: https://github.com/apache/incubator-druid/pull/8223 ### Description New SQL functions for filtering IPv4 addresses: - IPV4_MATCH: Check if IP address belongs to a subnet - IPV4_PARSE: Convert string IP

svn commit: r35103 - /dev/incubator/druid/0.15.1-incubating-rc2/

2019-08-01 Thread cwylie
Author: cwylie Date: Thu Aug 1 21:22:43 2019 New Revision: 35103 Log: add 0.15.1-incubating-rc2 artifacts Added: dev/incubator/druid/0.15.1-incubating-rc2/ dev/incubator/druid/0.15.1-incubating-rc2/apache-druid-0.15.1-incubating-bin.tar.gz (with props)

[GitHub] [incubator-druid] WilliamWhispell commented on issue #8221: useDefaultValueForNull not respected during compaction

2019-08-01 Thread GitBox
WilliamWhispell commented on issue #8221: useDefaultValueForNull not respected during compaction URL: https://github.com/apache/incubator-druid/issues/8221#issuecomment-517460605 I'm using s3 as a persistent store for the segments.. not sure if that has anything to do with it.

[GitHub] [incubator-druid] gianm commented on issue #8177: Update to Curator 4.2.0, ZooKeeper 3.4.14.

2019-08-01 Thread GitBox
gianm commented on issue #8177: Update to Curator 4.2.0, ZooKeeper 3.4.14. URL: https://github.com/apache/incubator-druid/pull/8177#issuecomment-517458152 > @gianm can you fix test failures? It's broken because of the upstream Curator issue that @asdf2014 linked.

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8210: Web-Console: add go to task for datasources

2019-08-01 Thread GitBox
vogievetsky commented on a change in pull request #8210: Web-Console: add go to task for datasources URL: https://github.com/apache/incubator-druid/pull/8210#discussion_r309894436 ## File path: web-console/src/views/task-view/tasks-view.tsx ## @@ -528,6 +544,15 @@ ORDER

[GitHub] [incubator-druid] capistrant commented on a change in pull request #7562: Enable toggling request logging on/off for different query types

2019-08-01 Thread GitBox
capistrant commented on a change in pull request #7562: Enable toggling request logging on/off for different query types URL: https://github.com/apache/incubator-druid/pull/7562#discussion_r309876602 ## File path:

[GitHub] [incubator-druid] capistrant commented on a change in pull request #7562: Enable toggling request logging on/off for different query types

2019-08-01 Thread GitBox
capistrant commented on a change in pull request #7562: Enable toggling request logging on/off for different query types URL: https://github.com/apache/incubator-druid/pull/7562#discussion_r309876081 ## File path:

[incubator-druid] annotated tag druid-0.15.1-incubating-rc2 updated (c698daa -> 7ff6420)

2019-08-01 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a change to annotated tag druid-0.15.1-incubating-rc2 in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. *** WARNING: tag druid-0.15.1-incubating-rc2 was modified! *** from c698daa

[incubator-druid] 02/02: [maven-release-plugin] prepare for next development iteration

2019-08-01 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch 0.15.1-incubating in repository https://gitbox.apache.org/repos/asf/incubator-druid.git commit e91de410f520f325599a9c59e505c6bb93931b40 Author: Clint Wylie AuthorDate: Thu Aug 1 12:58:16 2019

[incubator-druid] branch 0.15.1-incubating updated (b76ecc5 -> e91de41)

2019-08-01 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a change to branch 0.15.1-incubating in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. from b76ecc5 Fixed a bug in HttpPostEmitter leading to ClassCastException (#8205) (#8220) new

[incubator-druid] 01/02: [maven-release-plugin] prepare release druid-0.15.1-incubating-rc2

2019-08-01 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch 0.15.1-incubating in repository https://gitbox.apache.org/repos/asf/incubator-druid.git commit c698daab56a4b0612d6680b6359924653e938863 Author: Clint Wylie AuthorDate: Thu Aug 1 12:58:05 2019

[GitHub] [incubator-druid] arhantpanda commented on issue #8208: Druid SQL: ClassCastException: RelSubset cannot be cast to DruidRel

2019-08-01 Thread GitBox
arhantpanda commented on issue #8208: Druid SQL: ClassCastException: RelSubset cannot be cast to DruidRel URL: https://github.com/apache/incubator-druid/issues/8208#issuecomment-517427017 Similar to https://github.com/apache/incubator-druid/issues/4204#issuecomment-296917587

[incubator-druid] branch 0.15.1-incubating updated: Fixed a bug in HttpPostEmitter leading to ClassCastException (#8205) (#8220)

2019-08-01 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch 0.15.1-incubating in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/0.15.1-incubating by this push: new b76ecc5 Fixed a

[GitHub] [incubator-druid] clintropolis merged pull request #8220: Fix a bug in HttpPostEmitter leading to ClassCastException

2019-08-01 Thread GitBox
clintropolis merged pull request #8220: Fix a bug in HttpPostEmitter leading to ClassCastException URL: https://github.com/apache/incubator-druid/pull/8220 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-druid] clintropolis commented on issue #8220: Fix a bug in HttpPostEmitter leading to ClassCastException

2019-08-01 Thread GitBox
clintropolis commented on issue #8220: Fix a bug in HttpPostEmitter leading to ClassCastException URL: https://github.com/apache/incubator-druid/pull/8220#issuecomment-517423490 same timeout issue with integration test 1 as other backports, merging anyway

[GitHub] [incubator-druid] jihoonson opened a new pull request #8222: Fix bugs in overshadowableManager and add unit tests

2019-08-01 Thread GitBox
jihoonson opened a new pull request #8222: Fix bugs in overshadowableManager and add unit tests URL: https://github.com/apache/incubator-druid/pull/8222 ### Description Several bugs are found in the overshadowableManager. It could happen in various cases, but it was essentially the

[GitHub] [incubator-druid] egor-ryashin commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegment

2019-08-01 Thread GitBox
egor-ryashin commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments URL: https://github.com/apache/incubator-druid/pull/7306#discussion_r309847122 ## File path:

[GitHub] [incubator-druid] Fokko commented on issue #8138: Add SuppressWarnings SS_SHOULD_BE_STATIC

2019-08-01 Thread GitBox
Fokko commented on issue #8138: Add SuppressWarnings SS_SHOULD_BE_STATIC URL: https://github.com/apache/incubator-druid/pull/8138#issuecomment-517413703 That makes sense, I'll merge/pull in the future  This is an automated

[GitHub] [incubator-druid] leventov edited a comment on issue #8138: Add SuppressWarnings SS_SHOULD_BE_STATIC

2019-08-01 Thread GitBox
leventov edited a comment on issue #8138: Add SuppressWarnings SS_SHOULD_BE_STATIC URL: https://github.com/apache/incubator-druid/pull/8138#issuecomment-517413058 In Druid, the history is linear because of squash-and-merge as the merge action.

[GitHub] [incubator-druid] leventov commented on issue #8138: Add SuppressWarnings SS_SHOULD_BE_STATIC

2019-08-01 Thread GitBox
leventov commented on issue #8138: Add SuppressWarnings SS_SHOULD_BE_STATIC URL: https://github.com/apache/incubator-druid/pull/8138#issuecomment-517413058 In Druid, history is linear because of squash-and-merge as the merge action.

[GitHub] [incubator-druid] Fokko commented on issue #8138: Add SuppressWarnings SS_SHOULD_BE_STATIC

2019-08-01 Thread GitBox
Fokko commented on issue #8138: Add SuppressWarnings SS_SHOULD_BE_STATIC URL: https://github.com/apache/incubator-druid/pull/8138#issuecomment-517412709 I'm fine with merging as well. I usually do a pull instead `git pull github/master`, where github is my upstream. Most of the

[incubator-druid] branch master updated: Add IPv4 druid expressions (#8197)

2019-08-01 Thread gian
This is an automated email from the ASF dual-hosted git repository. gian pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/master by this push: new 7783b31 Add IPv4 druid expressions

[GitHub] [incubator-druid] gianm merged pull request #8197: Add IPv4 druid expressions

2019-08-01 Thread GitBox
gianm merged pull request #8197: Add IPv4 druid expressions URL: https://github.com/apache/incubator-druid/pull/8197 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #6974: sql support for dynamic parameters

2019-08-01 Thread GitBox
clintropolis commented on a change in pull request #6974: sql support for dynamic parameters URL: https://github.com/apache/incubator-druid/pull/6974#discussion_r309842970 ## File path: sql/src/test/java/org/apache/druid/sql/avatica/DruidAvaticaHandlerTest.java ## @@

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #6974: sql support for dynamic parameters

2019-08-01 Thread GitBox
clintropolis commented on a change in pull request #6974: sql support for dynamic parameters URL: https://github.com/apache/incubator-druid/pull/6974#discussion_r309843074 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/planner/PlannerContext.java ## @@

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #6974: sql support for dynamic parameters

2019-08-01 Thread GitBox
clintropolis commented on a change in pull request #6974: sql support for dynamic parameters URL: https://github.com/apache/incubator-druid/pull/6974#discussion_r309843040 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/planner/RelParameterizerShuttle.java ##

[GitHub] [incubator-druid] leventov commented on a change in pull request #8157: Enum of ResponseContext keys

2019-08-01 Thread GitBox
leventov commented on a change in pull request #8157: Enum of ResponseContext keys URL: https://github.com/apache/incubator-druid/pull/8157#discussion_r309841059 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -76,56

[GitHub] [incubator-druid] leventov commented on a change in pull request #8157: Enum of ResponseContext keys

2019-08-01 Thread GitBox
leventov commented on a change in pull request #8157: Enum of ResponseContext keys URL: https://github.com/apache/incubator-druid/pull/8157#discussion_r309839085 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -236,95

[GitHub] [incubator-druid] leventov commented on a change in pull request #8157: Enum of ResponseContext keys

2019-08-01 Thread GitBox
leventov commented on a change in pull request #8157: Enum of ResponseContext keys URL: https://github.com/apache/incubator-druid/pull/8157#discussion_r309826852 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -155,47

[GitHub] [incubator-druid] leventov commented on a change in pull request #8157: Enum of ResponseContext keys

2019-08-01 Thread GitBox
leventov commented on a change in pull request #8157: Enum of ResponseContext keys URL: https://github.com/apache/incubator-druid/pull/8157#discussion_r309835592 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -236,95

[GitHub] [incubator-druid] leventov commented on a change in pull request #8157: Enum of ResponseContext keys

2019-08-01 Thread GitBox
leventov commented on a change in pull request #8157: Enum of ResponseContext keys URL: https://github.com/apache/incubator-druid/pull/8157#discussion_r309837380 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -236,95

[GitHub] [incubator-druid] leventov commented on a change in pull request #8157: Enum of ResponseContext keys

2019-08-01 Thread GitBox
leventov commented on a change in pull request #8157: Enum of ResponseContext keys URL: https://github.com/apache/incubator-druid/pull/8157#discussion_r309838142 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -236,95

[GitHub] [incubator-druid] leventov commented on a change in pull request #8157: Enum of ResponseContext keys

2019-08-01 Thread GitBox
leventov commented on a change in pull request #8157: Enum of ResponseContext keys URL: https://github.com/apache/incubator-druid/pull/8157#discussion_r309830904 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -155,47

[GitHub] [incubator-druid] vogievetsky commented on issue #8219: Web-Console: stop log from making request after dialog close, implement query manager

2019-08-01 Thread GitBox
vogievetsky commented on issue #8219: Web-Console: stop log from making request after dialog close, implement query manager URL: https://github.com/apache/incubator-druid/pull/8219#issuecomment-517404211 Can some one tag this as 'Bug' please

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8219: Web-Console: stop log from making request after dialog close, implement query manager

2019-08-01 Thread GitBox
vogievetsky commented on a change in pull request #8219: Web-Console: stop log from making request after dialog close, implement query manager URL: https://github.com/apache/incubator-druid/pull/8219#discussion_r309834532 ## File path:

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8219: Web-Console: stop log from making request after dialog close, implement query manager

2019-08-01 Thread GitBox
vogievetsky commented on a change in pull request #8219: Web-Console: stop log from making request after dialog close, implement query manager URL: https://github.com/apache/incubator-druid/pull/8219#discussion_r309834792 ## File path:

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8219: Web-Console: stop log from making request after dialog close, implement query manager

2019-08-01 Thread GitBox
vogievetsky commented on a change in pull request #8219: Web-Console: stop log from making request after dialog close, implement query manager URL: https://github.com/apache/incubator-druid/pull/8219#discussion_r309834015 ## File path:

[incubator-druid-website] branch asf-site updated: Autobuild

2019-08-01 Thread fjy
This is an automated email from the ASF dual-hosted git repository. fjy pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-druid-website.git The following commit(s) were added to refs/heads/asf-site by this push: new 08460a7 Autobuild new

[GitHub] [incubator-druid-website] fjy merged pull request #27: Autobuild

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

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

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

[GitHub] [incubator-druid] WilliamWhispell opened a new issue #8221: useDefaultValueForNull not respected during compaction

2019-08-01 Thread GitBox
WilliamWhispell opened a new issue #8221: useDefaultValueForNull not respected during compaction URL: https://github.com/apache/incubator-druid/issues/8221 Reproducer: 1. set druid.generic.useDefaultValueForNull=false in common.runtime.properties and start druid (micro-quick-start)

[GitHub] [incubator-druid] leventov commented on issue #8138: Add SuppressWarnings SS_SHOULD_BE_STATIC

2019-08-01 Thread GitBox
leventov commented on issue #8138: Add SuppressWarnings SS_SHOULD_BE_STATIC URL: https://github.com/apache/incubator-druid/pull/8138#issuecomment-517393957 You need to merge master into your branch, not rebase. I usually do it as follows: ``` $ git fetch upstream master $ git

[GitHub] [incubator-druid] leventov commented on a change in pull request #7562: Enable toggling request logging on/off for different query types

2019-08-01 Thread GitBox
leventov commented on a change in pull request #7562: Enable toggling request logging on/off for different query types URL: https://github.com/apache/incubator-druid/pull/7562#discussion_r309822740 ## File path:

[GitHub] [incubator-druid] leventov commented on a change in pull request #7562: Enable toggling request logging on/off for different query types

2019-08-01 Thread GitBox
leventov commented on a change in pull request #7562: Enable toggling request logging on/off for different query types URL: https://github.com/apache/incubator-druid/pull/7562#discussion_r309819790 ## File path: docs/content/configuration/index.md ## @@ -379,6 +379,7 @@

[GitHub] [incubator-druid] leventov commented on a change in pull request #7562: Enable toggling request logging on/off for different query types

2019-08-01 Thread GitBox
leventov commented on a change in pull request #7562: Enable toggling request logging on/off for different query types URL: https://github.com/apache/incubator-druid/pull/7562#discussion_r309822632 ## File path:

[GitHub] [incubator-druid] leventov commented on a change in pull request #7562: Enable toggling request logging on/off for different query types

2019-08-01 Thread GitBox
leventov commented on a change in pull request #7562: Enable toggling request logging on/off for different query types URL: https://github.com/apache/incubator-druid/pull/7562#discussion_r309822785 ## File path:

[GitHub] [incubator-druid] leventov commented on a change in pull request #7562: Enable toggling request logging on/off for different query types

2019-08-01 Thread GitBox
leventov commented on a change in pull request #7562: Enable toggling request logging on/off for different query types URL: https://github.com/apache/incubator-druid/pull/7562#discussion_r309822672 ## File path:

[GitHub] [incubator-druid] leventov commented on a change in pull request #7562: Enable toggling request logging on/off for different query types

2019-08-01 Thread GitBox
leventov commented on a change in pull request #7562: Enable toggling request logging on/off for different query types URL: https://github.com/apache/incubator-druid/pull/7562#discussion_r309821738 ## File path:

[GitHub] [incubator-druid] leventov commented on a change in pull request #7562: Enable toggling request logging on/off for different query types

2019-08-01 Thread GitBox
leventov commented on a change in pull request #7562: Enable toggling request logging on/off for different query types URL: https://github.com/apache/incubator-druid/pull/7562#discussion_r309821170 ## File path: docs/content/configuration/index.md ## @@ -379,6 +379,7 @@

[GitHub] [incubator-druid] leventov commented on a change in pull request #7562: Enable toggling request logging on/off for different query types

2019-08-01 Thread GitBox
leventov commented on a change in pull request #7562: Enable toggling request logging on/off for different query types URL: https://github.com/apache/incubator-druid/pull/7562#discussion_r309821899 ## File path:

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8047: optimize single input column multi-value expressions

2019-08-01 Thread GitBox
ccaominh commented on a change in pull request #8047: optimize single input column multi-value expressions URL: https://github.com/apache/incubator-druid/pull/8047#discussion_r309821049 ## File path: core/src/main/java/org/apache/druid/math/expr/Function.java ## @@

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #6974: sql support for dynamic parameters

2019-08-01 Thread GitBox
ccaominh commented on a change in pull request #6974: sql support for dynamic parameters URL: https://github.com/apache/incubator-druid/pull/6974#discussion_r309813676 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/planner/RelParameterizerShuttle.java ## @@

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #6974: sql support for dynamic parameters

2019-08-01 Thread GitBox
ccaominh commented on a change in pull request #6974: sql support for dynamic parameters URL: https://github.com/apache/incubator-druid/pull/6974#discussion_r309813099 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/planner/PlannerContext.java ## @@ -191,6

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #6974: sql support for dynamic parameters

2019-08-01 Thread GitBox
ccaominh commented on a change in pull request #6974: sql support for dynamic parameters URL: https://github.com/apache/incubator-druid/pull/6974#discussion_r309806597 ## File path:

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #6974: sql support for dynamic parameters

2019-08-01 Thread GitBox
ccaominh commented on a change in pull request #6974: sql support for dynamic parameters URL: https://github.com/apache/incubator-druid/pull/6974#discussion_r309816132 ## File path: sql/src/test/java/org/apache/druid/sql/avatica/DruidAvaticaHandlerTest.java ## @@ -900,6

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #6974: sql support for dynamic parameters

2019-08-01 Thread GitBox
ccaominh commented on a change in pull request #6974: sql support for dynamic parameters URL: https://github.com/apache/incubator-druid/pull/6974#discussion_r309808680 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/planner/DruidPlanner.java ## @@ -57,22

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #6974: sql support for dynamic parameters

2019-08-01 Thread GitBox
ccaominh commented on a change in pull request #6974: sql support for dynamic parameters URL: https://github.com/apache/incubator-druid/pull/6974#discussion_r309802667 ## File path: docs/content/querying/sql.md ## @@ -439,6 +442,17 @@ of configuration. You can make Druid

[GitHub] [incubator-druid] leventov opened a new pull request #8220: Fix a bug in HttpPostEmitter leading to ClassCastException

2019-08-01 Thread GitBox
leventov opened a new pull request #8220: Fix a bug in HttpPostEmitter leading to ClassCastException URL: https://github.com/apache/incubator-druid/pull/8220 Backport of #8205. This is an automated message from the

[incubator-druid] branch master updated: Fixed a bug in HttpPostEmitter leading to ClassCastException (#8205)

2019-08-01 Thread leventov
This is an automated email from the ASF dual-hosted git repository. leventov pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/master by this push: new bcaffe2 Fixed a bug in

[GitHub] [incubator-druid] leventov merged pull request #8205: Fixed a bug in HttpPostEmitter leading to ClassCastException

2019-08-01 Thread GitBox
leventov merged pull request #8205: Fixed a bug in HttpPostEmitter leading to ClassCastException URL: https://github.com/apache/incubator-druid/pull/8205 This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-druid] leventov closed issue #8204: HttpPostEmitter throw Class cast exception when using emitAndReturnBatch

2019-08-01 Thread GitBox
leventov closed issue #8204: HttpPostEmitter throw Class cast exception when using emitAndReturnBatch URL: https://github.com/apache/incubator-druid/issues/8204 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-druid] leventov commented on a change in pull request #8205: Fixed a bug in HttpPostEmitter leading to ClassCastException

2019-08-01 Thread GitBox
leventov commented on a change in pull request #8205: Fixed a bug in HttpPostEmitter leading to ClassCastException URL: https://github.com/apache/incubator-druid/pull/8205#discussion_r309815207 ## File path:

[GitHub] [incubator-druid] leventov commented on a change in pull request #8205: Fixed a bug in HttpPostEmitter leading to ClassCastException

2019-08-01 Thread GitBox
leventov commented on a change in pull request #8205: Fixed a bug in HttpPostEmitter leading to ClassCastException URL: https://github.com/apache/incubator-druid/pull/8205#discussion_r309815207 ## File path:

[GitHub] [incubator-druid] leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments

2019-08-01 Thread GitBox
leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments URL: https://github.com/apache/incubator-druid/pull/7306#discussion_r309812489 ## File path:

[GitHub] [incubator-druid] leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments

2019-08-01 Thread GitBox
leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments URL: https://github.com/apache/incubator-druid/pull/7306#discussion_r309811605 ## File path:

[GitHub] [incubator-druid] mcbrewster opened a new pull request #8219: Web-Console: stop log from making request after dialog close, implement query manager

2019-08-01 Thread GitBox
mcbrewster opened a new pull request #8219: Web-Console: stop log from making request after dialog close, implement query manager URL: https://github.com/apache/incubator-druid/pull/8219 https://user-images.githubusercontent.com/37322608/62313908-b5a96a00-b446-11e9-9aa7-dc205a01f03a.png;>

[GitHub] [incubator-druid] leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments

2019-08-01 Thread GitBox
leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments URL: https://github.com/apache/incubator-druid/pull/7306#discussion_r309803728 ## File path:

[GitHub] [incubator-druid] leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments

2019-08-01 Thread GitBox
leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments URL: https://github.com/apache/incubator-druid/pull/7306#discussion_r309802264 ## File path:

[GitHub] [incubator-druid] leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments

2019-08-01 Thread GitBox
leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments URL: https://github.com/apache/incubator-druid/pull/7306#discussion_r309801369 ## File path:

[GitHub] [incubator-druid] leventov commented on issue #4349: [Proposal] Improve Handling of Nulls in Druid

2019-08-01 Thread GitBox
leventov commented on issue #4349: [Proposal] Improve Handling of Nulls in Druid URL: https://github.com/apache/incubator-druid/issues/4349#issuecomment-517371456 @WilliamWhispell there is nothing wrong with this line. `emptyToNullIfNeeded()` method may be misused somewhere, but the

  1   2   >