[GitHub] [incubator-druid] stale[bot] closed pull request #6493: Support for request logging in kafka emitter.

2019-04-12 Thread GitBox
stale[bot] closed pull request #6493: Support for request logging in kafka emitter. URL: https://github.com/apache/incubator-druid/pull/6493 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-druid] stale[bot] commented on issue #5739: make statsd emitter extensible

2019-04-12 Thread GitBox
stale[bot] commented on issue #5739: make statsd emitter extensible URL: https://github.com/apache/incubator-druid/pull/5739#issuecomment-482776802 This pull request has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive

[GitHub] [incubator-druid] stale[bot] closed pull request #5739: make statsd emitter extensible

2019-04-12 Thread GitBox
stale[bot] closed pull request #5739: make statsd emitter extensible URL: https://github.com/apache/incubator-druid/pull/5739 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [incubator-druid] stale[bot] commented on issue #6493: Support for request logging in kafka emitter.

2019-04-12 Thread GitBox
stale[bot] commented on issue #6493: Support for request logging in kafka emitter. URL: https://github.com/apache/incubator-druid/pull/6493#issuecomment-482776801 This pull request has been closed due to lack of activity. If you think that is incorrect, or the pull request requires

[GitHub] [incubator-druid] jon-wei edited a comment on issue #6972: Support LDAP authentication/authorization

2019-04-12 Thread GitBox
jon-wei edited a comment on issue #6972: Support LDAP authentication/authorization URL: https://github.com/apache/incubator-druid/pull/6972#issuecomment-482765704 @mohammadjkhan > LDAP is supposed to be source of truth for users/groups and not roles/permissions. The standard way

[GitHub] [incubator-druid] jon-wei edited a comment on issue #6972: Support LDAP authentication/authorization

2019-04-12 Thread GitBox
jon-wei edited a comment on issue #6972: Support LDAP authentication/authorization URL: https://github.com/apache/incubator-druid/pull/6972#issuecomment-482765704 @mohammadjkhan > LDAP is supposed to be source of truth for users/groups and not roles/permissions. The standard way

[GitHub] [incubator-druid] jon-wei edited a comment on issue #6972: Support LDAP authentication/authorization

2019-04-12 Thread GitBox
jon-wei edited a comment on issue #6972: Support LDAP authentication/authorization URL: https://github.com/apache/incubator-druid/pull/6972#issuecomment-482765704 @mohammadjkhan > LDAP is supposed to be source of truth for users/groups and not roles/permissions. The standard way

[incubator-druid] branch master updated: 7153: Isolated a supplier's close by removing its invocation (#7459)

2019-04-12 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 c57e62b 7153: Isolated a supplier's

[GitHub] [incubator-druid] gianm merged pull request #7459: Fix Kafka consumer non-thread-safe close

2019-04-12 Thread GitBox
gianm merged pull request #7459: Fix Kafka consumer non-thread-safe close URL: https://github.com/apache/incubator-druid/pull/7459 This is an automated message from the Apache Git Service. To respond to the message, please

[incubator-druid] branch master updated: Support querying realtime segments using time-ordered scan queries and fix broken scan queries without time column (#7454)

2019-04-12 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 85f10ed Support querying realtime

[GitHub] [incubator-druid] gianm merged pull request #7454: Support querying realtime segments using time-ordered scan queries and fix broken scan queries without time column

2019-04-12 Thread GitBox
gianm merged pull request #7454: Support querying realtime segments using time-ordered scan queries and fix broken scan queries without time column URL: https://github.com/apache/incubator-druid/pull/7454 This is an

[incubator-druid] branch master updated: Adjust BufferAggregator.get() impls to return copies (#7464)

2019-04-12 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 7d9cb69 Adjust BufferAggregator.get()

[GitHub] [incubator-druid] gianm merged pull request #7464: Adjust BufferAggregator.get() impls to return copies

2019-04-12 Thread GitBox
gianm merged pull request #7464: Adjust BufferAggregator.get() impls to return copies URL: https://github.com/apache/incubator-druid/pull/7464 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-druid] gianm closed issue #7461: HLLSketchBuild: Intermittent errors at query time

2019-04-12 Thread GitBox
gianm closed issue #7461: HLLSketchBuild: Intermittent errors at query time URL: https://github.com/apache/incubator-druid/issues/7461 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-druid] jon-wei commented on issue #6972: Support LDAP authentication/authorization

2019-04-12 Thread GitBox
jon-wei commented on issue #6972: Support LDAP authentication/authorization URL: https://github.com/apache/incubator-druid/pull/6972#issuecomment-482765704 @mohammadjkhan > LDAP is supposed to be source of truth for users/groups and not roles/permissions. The standard way to

[GitHub] [incubator-druid] leerho commented on issue #7461: HLLSketchBuild: Intermittent errors at query time

2019-04-12 Thread GitBox
leerho commented on issue #7461: HLLSketchBuild: Intermittent errors at query time URL: https://github.com/apache/incubator-druid/issues/7461#issuecomment-482765147 @gianm We have not. Nevertheless, I'm sure you know this, but these sketches are not thread-safe. I hope that

[incubator-druid] branch master updated: Some docs optimization (#6890)

2019-04-12 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 1d9450d Some docs optimization

[GitHub] [incubator-druid] jon-wei merged pull request #6890: Some docs optimization

2019-04-12 Thread GitBox
jon-wei merged pull request #6890: Some docs optimization URL: https://github.com/apache/incubator-druid/pull/6890 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [incubator-druid] surekhasaharan commented on issue #6834: [Proposal] Add published segment cache in broker

2019-04-12 Thread GitBox
surekhasaharan commented on issue #6834: [Proposal] Add published segment cache in broker URL: https://github.com/apache/incubator-druid/issues/6834#issuecomment-482755973 Binary protocol was not considered at the time, but it does make sense to replace json with smile format for all

[GitHub] [incubator-druid] jihoonson commented on issue #7423: Unused parameter in ParallelIndexTuningConfig

2019-04-12 Thread GitBox
jihoonson commented on issue #7423: Unused parameter in ParallelIndexTuningConfig URL: https://github.com/apache/incubator-druid/issues/7423#issuecomment-482754776 Hmm, isn't it being used in `ParallelIndexSubTask.run()`?

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #7428: Add errors and state to stream supervisor status API endpoint

2019-04-12 Thread GitBox
jon-wei commented on a change in pull request #7428: Add errors and state to stream supervisor status API endpoint URL: https://github.com/apache/incubator-druid/pull/7428#discussion_r275091952 ## File path: docs/content/development/extensions-core/kafka-ingestion.md ##

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #7428: Add errors and state to stream supervisor status API endpoint

2019-04-12 Thread GitBox
jon-wei commented on a change in pull request #7428: Add errors and state to stream supervisor status API endpoint URL: https://github.com/apache/incubator-druid/pull/7428#discussion_r275091559 ## File path: docs/content/development/extensions-core/kafka-ingestion.md ##

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r275073458 ## File path: server/src/main/java/org/apache/druid/server/http/MetadataResource.java

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r275090562 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/schema/SystemSchema.java

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r275090369 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/schema/SystemSchema.java

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r275090186 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/schema/SystemSchema.java

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r275076806 ## File path: server/src/main/java/org/apache/druid/server/http/MetadataResource.java

[GitHub] [incubator-druid] yurmix commented on issue #4032: [Proposal] Simple join support

2019-04-12 Thread GitBox
yurmix commented on issue #4032: [Proposal] Simple join support URL: https://github.com/apache/incubator-druid/issues/4032#issuecomment-482753395 > @yurmix for multi-value lookups, would you be thinking about extending the lookup framework to support multiple values for the same key? Or

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r275074252 ## File path: server/src/main/java/org/apache/druid/server/http/MetadataResource.java

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r275063339 ## File path: core/src/main/java/org/apache/druid/timeline/SegmentWithOvershadowedStatus.java

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r275089539 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/schema/MetadataSegmentView.java

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r275088852 ## File path: server/src/main/java/org/apache/druid/server/http/MetadataResource.java

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r275088705 ## File path: server/src/main/java/org/apache/druid/server/http/MetadataResource.java

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r275066881 ## File path: server/src/main/java/org/apache/druid/server/http/MetadataResource.java

[GitHub] [incubator-druid] jon-wei commented on issue #7464: Adjust BufferAggregator.get() impls to return copies

2019-04-12 Thread GitBox
jon-wei commented on issue #7464: Adjust BufferAggregator.get() impls to return copies URL: https://github.com/apache/incubator-druid/pull/7464#issuecomment-482749937 I updated this to use `compact()` and added a note on mutability of return objects for BufferAggregator.get()

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #7464: Adjust BufferAggregator.get() impls to return copies

2019-04-12 Thread GitBox
jon-wei commented on a change in pull request #7464: Adjust BufferAggregator.get() impls to return copies URL: https://github.com/apache/incubator-druid/pull/7464#discussion_r275088332 ## File path: processing/src/main/java/org/apache/druid/query/aggregation/BufferAggregator.java

[GitHub] [incubator-druid] leerho commented on issue #6814: [Discuss] Replacing hyperUnique as 'default' distinct count sketch

2019-04-12 Thread GitBox
leerho commented on issue #6814: [Discuss] Replacing hyperUnique as 'default' distinct count sketch URL: https://github.com/apache/incubator-druid/issues/6814#issuecomment-482747649 @Dylan1312 Ok, this is helpful, and thank you for performing these experiments! 1) The Druid HLL

[GitHub] [incubator-druid] gianm commented on a change in pull request #7464: Adjust BufferAggregator.get() impls to return copies

2019-04-12 Thread GitBox
gianm commented on a change in pull request #7464: Adjust BufferAggregator.get() impls to return copies URL: https://github.com/apache/incubator-druid/pull/7464#discussion_r275078019 ## File path: processing/src/main/java/org/apache/druid/query/aggregation/BufferAggregator.java

[GitHub] [incubator-druid] gianm commented on issue #7464: Adjust BufferAggregator.get() impls to return copies

2019-04-12 Thread GitBox
gianm commented on issue #7464: Adjust BufferAggregator.get() impls to return copies URL: https://github.com/apache/incubator-druid/pull/7464#issuecomment-482737478 It seems like `compact()` would work on UpdateDoublesSketch.

[GitHub] [incubator-druid] jon-wei opened a new pull request #7464: Adjust BufferAggregator.get() impls to return copies

2019-04-12 Thread GitBox
jon-wei opened a new pull request #7464: Adjust BufferAggregator.get() impls to return copies URL: https://github.com/apache/incubator-druid/pull/7464 Fixes #7461 This PR adjusts `HllSketchBuildBufferAggregator` and `BaseBloomFilterBufferAggregator` to return on-heap copies of the

[GitHub] [incubator-druid] egor-ryashin edited a comment on issue #6751: Calcite column resolution bug

2019-04-12 Thread GitBox
egor-ryashin edited a comment on issue #6751: Calcite column resolution bug URL: https://github.com/apache/incubator-druid/issues/6751#issuecomment-482724282 ~@vogievetsky I wonder if it still persists? couldn't reproduce it straight away.~ Reproduced (it wasn't reproduced in

[GitHub] [incubator-druid] shuqi7 commented on a change in pull request #7460: Add created_time as tie breaker when sorting by status in task table

2019-04-12 Thread GitBox
shuqi7 commented on a change in pull request #7460: Add created_time as tie breaker when sorting by status in task table URL: https://github.com/apache/incubator-druid/pull/7460#discussion_r275072717 ## File path: web-console/src/views/tasks-view.tsx ## @@ -521,9 +521,10

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #7460: Add created_time as tie breaker when sorting by status in task table

2019-04-12 Thread GitBox
vogievetsky commented on a change in pull request #7460: Add created_time as tie breaker when sorting by status in task table URL: https://github.com/apache/incubator-druid/pull/7460#discussion_r275072341 ## File path: web-console/src/views/tasks-view.tsx ## @@ -521,9

[GitHub] [incubator-druid] vogievetsky commented on issue #7463: Display null in SQL table cell

2019-04-12 Thread GitBox
vogievetsky commented on issue #7463: Display null in SQL table cell URL: https://github.com/apache/incubator-druid/pull/7463#issuecomment-482731045 This looks great,  thank you for this change. This is an automated message

[GitHub] [incubator-druid] Dylan1312 edited a comment on issue #6814: [Discuss] Replacing hyperUnique as 'default' distinct count sketch

2019-04-12 Thread GitBox
Dylan1312 edited a comment on issue #6814: [Discuss] Replacing hyperUnique as 'default' distinct count sketch URL: https://github.com/apache/incubator-druid/issues/6814#issuecomment-482727132 Hi Lee, Thanks for the response! - I've tried various configurations, each of

[GitHub] [incubator-druid] Dylan1312 commented on issue #6814: [Discuss] Replacing hyperUnique as 'default' distinct count sketch

2019-04-12 Thread GitBox
Dylan1312 commented on issue #6814: [Discuss] Replacing hyperUnique as 'default' distinct count sketch URL: https://github.com/apache/incubator-druid/issues/6814#issuecomment-482727132 Hi Lee, Thanks for the response! - I've tried various configurations, each of HLL4,6&8 and

[GitHub] [incubator-druid] gianm commented on a change in pull request #7454: Support querying realtime segments using time-ordered scan queries and fix broken scan queries without time column

2019-04-12 Thread GitBox
gianm commented on a change in pull request #7454: Support querying realtime segments using time-ordered scan queries and fix broken scan queries without time column URL: https://github.com/apache/incubator-druid/pull/7454#discussion_r275066537 ## File path:

[GitHub] [incubator-druid] gianm commented on a change in pull request #7454: Support querying realtime segments using time-ordered scan queries and fix broken scan queries without time column

2019-04-12 Thread GitBox
gianm commented on a change in pull request #7454: Support querying realtime segments using time-ordered scan queries and fix broken scan queries without time column URL: https://github.com/apache/incubator-druid/pull/7454#discussion_r275066537 ## File path:

[GitHub] [incubator-druid] egor-ryashin commented on issue #6751: Calcite column resolution bug

2019-04-12 Thread GitBox
egor-ryashin commented on issue #6751: Calcite column resolution bug URL: https://github.com/apache/incubator-druid/issues/6751#issuecomment-482724282 @vogievetsky I wonder if it still persists? couldn't reproduce it straight away.

[GitHub] [incubator-druid] gianm commented on a change in pull request #7454: Support querying realtime segments using time-ordered scan queries and fix broken scan queries without time column

2019-04-12 Thread GitBox
gianm commented on a change in pull request #7454: Support querying realtime segments using time-ordered scan queries and fix broken scan queries without time column URL: https://github.com/apache/incubator-druid/pull/7454#discussion_r275064400 ## File path:

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #7463: Display null in SQL table cell

2019-04-12 Thread GitBox
vogievetsky commented on a change in pull request #7463: Display null in SQL table cell URL: https://github.com/apache/incubator-druid/pull/7463#discussion_r275063979 ## File path: web-console/src/views/sql-view.tsx ## @@ -195,7 +195,17 @@ export class SqlView extends

[GitHub] [incubator-druid] shuqi7 commented on a change in pull request #7460: Add created_time as tie breaker when sorting by status in task table

2019-04-12 Thread GitBox
shuqi7 commented on a change in pull request #7460: Add created_time as tie breaker when sorting by status in task table URL: https://github.com/apache/incubator-druid/pull/7460#discussion_r275063932 ## File path: web-console/src/views/tasks-view.tsx ## @@ -521,9 +521,10

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #7460: Add created_time as tie breaker when sorting by status in task table

2019-04-12 Thread GitBox
vogievetsky commented on a change in pull request #7460: Add created_time as tie breaker when sorting by status in task table URL: https://github.com/apache/incubator-druid/pull/7460#discussion_r275063018 ## File path: web-console/src/views/tasks-view.tsx ## @@ -521,9

[GitHub] [incubator-druid] gianm commented on a change in pull request #7454: Support querying realtime segments using time-ordered scan queries and fix broken scan queries without time column

2019-04-12 Thread GitBox
gianm commented on a change in pull request #7454: Support querying realtime segments using time-ordered scan queries and fix broken scan queries without time column URL: https://github.com/apache/incubator-druid/pull/7454#discussion_r275059061 ## File path:

[GitHub] [incubator-druid] shuqi7 opened a new pull request #7463: Console sql table

2019-04-12 Thread GitBox
shuqi7 opened a new pull request #7463: Console sql table URL: https://github.com/apache/incubator-druid/pull/7463 - Originally if there is no data or an empty string in the table cell, it will just leave a blank cell - Now an italic _`null`_ will be displayed instead

[GitHub] [incubator-druid] shuqi7 opened a new pull request #7462: Capitalize keywords in SQL auto complete

2019-04-12 Thread GitBox
shuqi7 opened a new pull request #7462: Capitalize keywords in SQL auto complete URL: https://github.com/apache/incubator-druid/pull/7462 - All keywords are in upper case to follow SQL's convention

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #7145: Enhance the Http Firehose to work with URIs requiring basic authentication

2019-04-12 Thread GitBox
jihoonson commented on a change in pull request #7145: Enhance the Http Firehose to work with URIs requiring basic authentication URL: https://github.com/apache/incubator-druid/pull/7145#discussion_r275053244 ## File path:

[GitHub] [incubator-druid] leerho edited a comment on issue #6814: [Discuss] Replacing hyperUnique as 'default' distinct count sketch

2019-04-12 Thread GitBox
leerho edited a comment on issue #6814: [Discuss] Replacing hyperUnique as 'default' distinct count sketch URL: https://github.com/apache/incubator-druid/issues/6814#issuecomment-482710359 @Dylan1312 Sorry, I didn't see your comment until just now. I would be very interested in

[GitHub] [incubator-druid] leerho commented on issue #6814: [Discuss] Replacing hyperUnique as 'default' distinct count sketch

2019-04-12 Thread GitBox
leerho commented on issue #6814: [Discuss] Replacing hyperUnique as 'default' distinct count sketch URL: https://github.com/apache/incubator-druid/issues/6814#issuecomment-482710359 @Dylan1312 Sorry, I didn't see your comment until just now. I would be very interested in

[GitHub] [incubator-druid] gianm commented on issue #7461: HLLSketchBuild: Intermittent errors at query time

2019-04-12 Thread GitBox
gianm commented on issue #7461: HLLSketchBuild: Intermittent errors at query time URL: https://github.com/apache/incubator-druid/issues/7461#issuecomment-482709590 I think this is happening because HllSketchBuildBufferAggregator's `get(buf, position)` method does not copy data out from

[GitHub] [incubator-druid] gianm opened a new issue #7461: HLLSketchBuild: Intermittent errors at query time

2019-04-12 Thread GitBox
gianm opened a new issue #7461: HLLSketchBuild: Intermittent errors at query time URL: https://github.com/apache/incubator-druid/issues/7461 ### Affected Version Observed in Druid 0.13.0-incubating-iap9, an Imply build, but one that has no differences from 0.13.0-incubating in the

[GitHub] [incubator-druid] leventov commented on issue #7458: New Coordinator segment balancing/loading algorithm

2019-04-12 Thread GitBox
leventov commented on issue #7458: New Coordinator segment balancing/loading algorithm URL: https://github.com/apache/incubator-druid/issues/7458#issuecomment-482689404 > There was discussion previously about what can/should be done in an extension vs what must be done in core druid.

[GitHub] [incubator-druid] samarthjain commented on a change in pull request #7088: Improve parallelism of zookeeper based segment change processing

2019-04-12 Thread GitBox
samarthjain commented on a change in pull request #7088: Improve parallelism of zookeeper based segment change processing URL: https://github.com/apache/incubator-druid/pull/7088#discussion_r275021326 ## File path: docs/content/configuration/index.md ## @@ -1251,7

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #7350: Remove unnecessary collection

2019-04-12 Thread GitBox
surekhasaharan commented on a change in pull request #7350: Remove unnecessary collection URL: https://github.com/apache/incubator-druid/pull/7350#discussion_r275020823 ## File path: server/src/main/java/org/apache/druid/server/http/MetadataResource.java ## @@ -161,14

[GitHub] [incubator-druid] gianm commented on a change in pull request #7325: Fix result-level cache for queries

2019-04-12 Thread GitBox
gianm commented on a change in pull request #7325: Fix result-level cache for queries URL: https://github.com/apache/incubator-druid/pull/7325#discussion_r275016341 ## File path: processing/src/main/java/org/apache/druid/query/groupby/having/GreaterThanHavingSpec.java ##

[GitHub] [incubator-druid] emrecanagah commented on issue #7437: [restore data from metadata store] is it possible ? druid version 0.13

2019-04-12 Thread GitBox
emrecanagah commented on issue #7437: [restore data from metadata store] is it possible ? druid version 0.13 URL: https://github.com/apache/incubator-druid/issues/7437#issuecomment-482675286 I have been able to drop these segments from metadata store by using this API:

[GitHub] [incubator-druid] emrecanagah closed issue #7437: [restore data from metadata store] is it possible ? druid version 0.13

2019-04-12 Thread GitBox
emrecanagah closed issue #7437: [restore data from metadata store] is it possible ? druid version 0.13 URL: https://github.com/apache/incubator-druid/issues/7437 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #7325: Fix result-level cache for queries

2019-04-12 Thread GitBox
surekhasaharan commented on a change in pull request #7325: Fix result-level cache for queries URL: https://github.com/apache/incubator-druid/pull/7325#discussion_r275015283 ## File path: processing/src/main/java/org/apache/druid/query/groupby/having/GreaterThanHavingSpec.java

[GitHub] [incubator-druid] shuqi7 opened a new pull request #7460: Add created_time as tie breaker when sorting by status in task table

2019-04-12 Thread GitBox
shuqi7 opened a new pull request #7460: Add created_time as tie breaker when sorting by status in task table URL: https://github.com/apache/incubator-druid/pull/7460 - Rows with later created_time will be displayed on top when status of the rows are tied, regardless if the rows are sorted

[GitHub] [incubator-druid] drcrallen commented on issue #7458: New Coordinator segment balancing/loading algorithm

2019-04-12 Thread GitBox
drcrallen commented on issue #7458: New Coordinator segment balancing/loading algorithm URL: https://github.com/apache/incubator-druid/issues/7458#issuecomment-482648760 There was discussion previously about what can/should be done in an extension vs what must be done in core druid. Can

[GitHub] [incubator-druid] esevastyanov opened a new pull request #7459: Fix Kafka consumer non-thread-safe close

2019-04-12 Thread GitBox
esevastyanov opened a new pull request #7459: Fix Kafka consumer non-thread-safe close URL: https://github.com/apache/incubator-druid/pull/7459 #7153 Isolated a supplier's close by removing its invocation from a non-notice-processing thread. There were two `recordSupplier`'s stop call:

[GitHub] [incubator-druid] egor-ryashin commented on issue #6759: Unknown provider[mysql] of Key[type=org.apache.druid.metadata.MetadataStorageProvider,

2019-04-12 Thread GitBox
egor-ryashin commented on issue #6759: Unknown provider[mysql] of Key[type=org.apache.druid.metadata.MetadataStorageProvider, URL: https://github.com/apache/incubator-druid/issues/6759#issuecomment-482636979 > known options[[derby]] it says it didn't load the `mysql` extension,

[GitHub] [incubator-druid] leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r274968940 ## File path: server/src/main/java/org/apache/druid/server/http/MetadataResource.java ##

[GitHub] [incubator-druid] leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r274970775 ## File path: server/src/main/java/org/apache/druid/server/http/MetadataResource.java ##

[GitHub] [incubator-druid] leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r274972691 ## File path: server/src/main/java/org/apache/druid/server/http/MetadataResource.java ##

[GitHub] [incubator-druid] leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r274975101 ## File path: server/src/main/java/org/apache/druid/server/http/MetadataResource.java ##

[GitHub] [incubator-druid] leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r274973744 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/schema/MetadataSegmentView.java

[GitHub] [incubator-druid] leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r274973097 ## File path: server/src/main/java/org/apache/druid/server/http/MetadataResource.java ##

[GitHub] [incubator-druid] leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r274969871 ## File path: server/src/main/java/org/apache/druid/server/http/MetadataResource.java ##

[GitHub] [incubator-druid] leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r274971417 ## File path: server/src/main/java/org/apache/druid/server/http/MetadataResource.java ##

[GitHub] [incubator-druid] leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r274971644 ## File path: server/src/main/java/org/apache/druid/server/http/MetadataResource.java ##

[GitHub] [incubator-druid] leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r274975274 ## File path: core/src/main/java/org/apache/druid/timeline/SegmentWithOvershadowedStatus.java

[GitHub] [incubator-druid] leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r274972820 ## File path: server/src/main/java/org/apache/druid/server/http/MetadataResource.java ##

[GitHub] [incubator-druid] leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r274970136 ## File path: server/src/main/java/org/apache/druid/server/http/MetadataResource.java ##

[GitHub] [incubator-druid] leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

2019-04-12 Thread GitBox
leventov commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r274968520 ## File path: server/src/main/java/org/apache/druid/server/http/MetadataResource.java ##

[GitHub] [incubator-druid] leventov commented on issue #6834: [Proposal] Add published segment cache in broker

2019-04-12 Thread GitBox
leventov commented on issue #6834: [Proposal] Add published segment cache in broker URL: https://github.com/apache/incubator-druid/issues/6834#issuecomment-482636258 In addition to previous questions: was it considered to use binary communication protocol instead of JSON?

[GitHub] [incubator-druid] egor-ryashin commented on issue #6763: java.lang.NoClassDefFoundError

2019-04-12 Thread GitBox
egor-ryashin commented on issue #6763: java.lang.NoClassDefFoundError URL: https://github.com/apache/incubator-druid/issues/6763#issuecomment-482634353 @huahuang12 I wonder whether you have managed to resolve the issue? This

[GitHub] [incubator-druid] mohammadjkhan commented on issue #6972: Support LDAP authentication/authorization

2019-04-12 Thread GitBox
mohammadjkhan commented on issue #6972: Support LDAP authentication/authorization URL: https://github.com/apache/incubator-druid/pull/6972#issuecomment-482629784 @nishantmonu51 @jon-wei To summarize some of the comments and feedbacks in this PR, and to establish a

[GitHub] [incubator-druid] fjy merged pull request #7449: Wrap query with limit within the web console

2019-04-12 Thread GitBox
fjy merged pull request #7449: Wrap query with limit within the web console URL: https://github.com/apache/incubator-druid/pull/7449 This is an automated message from the Apache Git Service. To respond to the message, please

[incubator-druid] branch master updated: Wrap query with limit within the web console (#7449)

2019-04-12 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 60dd75d Wrap query with limit within the

[GitHub] [incubator-druid] leventov closed issue #6826: Concurrency bug in DruidSchema.refreshSegmentsForDataSource()

2019-04-12 Thread GitBox
leventov closed issue #6826: Concurrency bug in DruidSchema.refreshSegmentsForDataSource() URL: https://github.com/apache/incubator-druid/issues/6826 This is an automated message from the Apache Git Service. To respond to

[incubator-druid] branch master updated: Ensuring that all the controls are aligned correctly (#7457)

2019-04-12 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 3aae4aa Ensuring that all the controls

[GitHub] [incubator-druid] fjy merged pull request #7457: Ensuring that all the controls are aligned correctly

2019-04-12 Thread GitBox
fjy merged pull request #7457: Ensuring that all the controls are aligned correctly URL: https://github.com/apache/incubator-druid/pull/7457 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-druid] leventov commented on a change in pull request #7350: Remove unnecessary collection

2019-04-12 Thread GitBox
leventov commented on a change in pull request #7350: Remove unnecessary collection URL: https://github.com/apache/incubator-druid/pull/7350#discussion_r274965983 ## File path: server/src/main/java/org/apache/druid/server/http/MetadataResource.java ## @@ -161,14 +153,11

[GitHub] [incubator-druid] leventov opened a new issue #7458: New Coordinator segment balancing/loading algorithm

2019-04-12 Thread GitBox
leventov opened a new issue #7458: New Coordinator segment balancing/loading algorithm URL: https://github.com/apache/incubator-druid/issues/7458 ### Motivation and goal Currently, there is no segment balancing/loading algorithm. There is just code that happens to behave somehow,

[GitHub] [incubator-druid] egor-ryashin commented on issue #6776: How to migrate druid cluster?

2019-04-12 Thread GitBox
egor-ryashin commented on issue #6776: How to migrate druid cluster? URL: https://github.com/apache/incubator-druid/issues/6776#issuecomment-482572055 Related https://github.com/apache/incubator-druid/issues/5273 This is an

[GitHub] [incubator-druid] egor-ryashin closed issue #6779: KafkaIndexTask failed with IllegalArgumentException

2019-04-12 Thread GitBox
egor-ryashin closed issue #6779: KafkaIndexTask failed with IllegalArgumentException URL: https://github.com/apache/incubator-druid/issues/6779 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-druid] egor-ryashin commented on issue #6786: apache-druid-0.13.0-incubating router /druid/router/v1/brokers

2019-04-12 Thread GitBox
egor-ryashin commented on issue #6786: apache-druid-0.13.0-incubating router /druid/router/v1/brokers URL: https://github.com/apache/incubator-druid/issues/6786#issuecomment-482567151 @panqingcui could you post here Druid version and router configuration?

[GitHub] [incubator-druid] egor-ryashin commented on issue #6826: Concurrency bug in DruidSchema.refreshSegmentsForDataSource()

2019-04-12 Thread GitBox
egor-ryashin commented on issue #6826: Concurrency bug in DruidSchema.refreshSegmentsForDataSource() URL: https://github.com/apache/incubator-druid/issues/6826#issuecomment-482537642 @leventov I guess it's resolved? This is

[GitHub] [incubator-druid] egor-ryashin commented on issue #6925: Class org.apache.hadoop.hive.ql.io.orc.OrcNewSplit not found

2019-04-12 Thread GitBox
egor-ryashin commented on issue #6925: Class org.apache.hadoop.hive.ql.io.orc.OrcNewSplit not found URL: https://github.com/apache/incubator-druid/issues/6925#issuecomment-482497640 @chenhaiyan could you post here the task schema, the orc file schema and Druid version?

  1   2   >