[GitHub] [druid] successok commented on issue #11221: The task actually failed but the status shows success

2021-05-07 Thread GitBox
successok commented on issue #11221: URL: https://github.com/apache/druid/issues/11221#issuecomment-835123579 This is the new spec after reloading data from kafka which is normal:

[GitHub] [druid] successok edited a comment on issue #11221: The task actually failed but the status shows success

2021-05-07 Thread GitBox
successok edited a comment on issue #11221: URL: https://github.com/apache/druid/issues/11221#issuecomment-835113902 Here is the spec information: `{ "type": "kafka", "context": { "priority": 90, "forceTimeChunkLock": true }, "spec": {

[GitHub] [druid] successok edited a comment on issue #11221: The task actually failed but the status shows success

2021-05-07 Thread GitBox
successok edited a comment on issue #11221: URL: https://github.com/apache/druid/issues/11221#issuecomment-835113902 Here is the spec information: `{ "type": "kafka", "context": { "priority": 90, "forceTimeChunkLock": true }, "spec": {

[GitHub] [druid] successok edited a comment on issue #11221: The task actually failed but the status shows success

2021-05-07 Thread GitBox
successok edited a comment on issue #11221: URL: https://github.com/apache/druid/issues/11221#issuecomment-835113902 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For

[GitHub] [druid] successok commented on issue #11221: The task actually failed but the status shows success

2021-05-07 Thread GitBox
successok commented on issue #11221: URL: https://github.com/apache/druid/issues/11221#issuecomment-835113902 here is the spec information: `{ "type": "kafka", "context": { "priority": 90, "forceTimeChunkLock": true }, "spec": { "ioConfig": {

[GitHub] [druid] FrankChen021 commented on issue #11221: The task actually failed but the status shows success

2021-05-07 Thread GitBox
FrankChen021 commented on issue #11221: URL: https://github.com/apache/druid/issues/11221#issuecomment-835103686 The exception is thrown from KafkaEmitter, not the ingestion, so the ingest task does not fail. You could upload the entire task log so that we could know why there's such

[GitHub] [druid] successok opened a new issue #11221: The task actually failed but the status shows success

2021-05-07 Thread GitBox
successok opened a new issue #11221: URL: https://github.com/apache/druid/issues/11221 ### Affected Version the latest version 0.21.0 ### Description As a loyal user of druid, I upgraded the version immediately after it was upgraded to 0.21.0, When I created a datasource

[GitHub] [druid] FrankChen021 opened a new pull request #11220: Fix default value of druid.emitter.http.maxBatchSize in doc

2021-05-07 Thread GitBox
FrankChen021 opened a new pull request #11220: URL: https://github.com/apache/druid/pull/11220 ### Description According to the code, default value `druid.emitter.http.maxBatchSize` is 5 * 1024 * 1024, which is 5242880, while in the doc it's is 5191680.

[GitHub] [druid] loquisgon commented on a change in pull request #11123: Avoid memory mapping hydrants after they are persisted & after they are merged for native batch ingestion

2021-05-07 Thread GitBox
loquisgon commented on a change in pull request #11123: URL: https://github.com/apache/druid/pull/11123#discussion_r628624433 ## File path: server/pom.xml ## @@ -454,6 +454,11 @@ 1.3 test + Review comment: In next push --

[GitHub] [druid] loquisgon commented on a change in pull request #11123: Avoid memory mapping hydrants after they are persisted & after they are merged for native batch ingestion

2021-05-07 Thread GitBox
loquisgon commented on a change in pull request #11123: URL: https://github.com/apache/druid/pull/11123#discussion_r628624051 ## File path: docs/configuration/index.md ## @@ -1320,6 +1320,7 @@ Additional peon configs include: |`druid.peon.mode`|Choices are "local" and

[GitHub] [druid] loquisgon commented on a change in pull request #11123: Avoid memory mapping hydrants after they are persisted & after they are merged for native batch ingestion

2021-05-07 Thread GitBox
loquisgon commented on a change in pull request #11123: URL: https://github.com/apache/druid/pull/11123#discussion_r628623123 ## File path: server/src/main/java/org/apache/druid/segment/realtime/appenderator/AppenderatorImpl.java ## @@ -691,6 +739,8 @@ public Object call()

[GitHub] [druid] loquisgon commented on a change in pull request #11123: Avoid memory mapping hydrants after they are persisted & after they are merged for native batch ingestion

2021-05-07 Thread GitBox
loquisgon commented on a change in pull request #11123: URL: https://github.com/apache/druid/pull/11123#discussion_r628622779 ## File path: server/src/main/java/org/apache/druid/segment/realtime/appenderator/AppenderatorImpl.java ## @@ -164,13 +168,24 @@ private volatile

[GitHub] [druid] leerho commented on pull request #11201: Add "stringEncoding" parameter to DataSketches HLL.

2021-05-07 Thread GitBox
leerho commented on pull request #11201: URL: https://github.com/apache/druid/pull/11201#issuecomment-834830853 @gianm I want to clarify a comment made above. > HLL sketches used UTF-16LE encoding when hashing strings. This is not correct, at least for the HLL in

[GitHub] [druid] maytasm commented on a change in pull request #11123: Avoid memory mapping hydrants after they are persisted & after they are merged for native batch ingestion

2021-05-07 Thread GitBox
maytasm commented on a change in pull request #11123: URL: https://github.com/apache/druid/pull/11123#discussion_r628541779 ## File path: server/src/main/java/org/apache/druid/segment/realtime/appenderator/AppenderatorImpl.java ## @@ -164,13 +168,24 @@ private volatile

[GitHub] [druid] benkrug opened a new issue #11219: doubleMean ignores useDefaultValueForNull=false, treats nulls as zeros

2021-05-07 Thread GitBox
benkrug opened a new issue #11219: URL: https://github.com/apache/druid/issues/11219 ### Affected Version Seen in 0.20.0 ### Description doubleMean ignores the setting useDefaultValueForNull=false, and treats nulls as zeros. Taking a mean, it possibly ignores

[druid] branch master updated (cf2cde1 -> 2df4214)

2021-05-07 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/druid.git. from cf2cde1 add links to release notes, light refactor of landing page (#11051) add 2df4214 Fix idempotence of

[GitHub] [druid] jihoonson merged pull request #11189: Fix idempotence of segment allocation and task report apis in native batch ingestion

2021-05-07 Thread GitBox
jihoonson merged pull request #11189: URL: https://github.com/apache/druid/pull/11189 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service,

[GitHub] [druid] jihoonson commented on pull request #11189: Fix idempotence of segment allocation and task report apis in native batch ingestion

2021-05-07 Thread GitBox
jihoonson commented on pull request #11189: URL: https://github.com/apache/druid/pull/11189#issuecomment-834793112 @jon-wei @clintropolis thanks for the review! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[druid] branch master updated (3b9dad4 -> cf2cde1)

2021-05-07 Thread shetland
This is an automated email from the ASF dual-hosted git repository. shetland pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/druid.git. from 3b9dad4 Consolidate the number of Dockerfiles (#11187) add cf2cde1 add links to release notes, light

[GitHub] [druid] sthetland merged pull request #11051: add links to release notes, light refactor of landing page

2021-05-07 Thread GitBox
sthetland merged pull request #11051: URL: https://github.com/apache/druid/pull/11051 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service,

[GitHub] [druid] ericxiao251 commented on issue #10559: Query planner fails to plan nested query when an inner query is to topN and outer one has a filter

2021-05-07 Thread GitBox
ericxiao251 commented on issue #10559: URL: https://github.com/apache/druid/issues/10559#issuecomment-834786188 Hey @nishantmonu51 can I take this issue? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [druid] clintropolis commented on a change in pull request #11188: SQL timeseries no longer skip empty buckets with all granularity

2021-05-07 Thread GitBox
clintropolis commented on a change in pull request #11188: URL: https://github.com/apache/druid/pull/11188#discussion_r628489278 ## File path: docs/querying/sql.md ## @@ -293,15 +293,15 @@ the `UNNEST` functionality available in some other SQL dialects. Refer to the do The

[GitHub] [druid] clintropolis commented on a change in pull request #11188: SQL timeseries no longer skip empty buckets with all granularity

2021-05-07 Thread GitBox
clintropolis commented on a change in pull request #11188: URL: https://github.com/apache/druid/pull/11188#discussion_r628487653 ## File path: docs/querying/sql.md ## @@ -293,15 +293,15 @@ the `UNNEST` functionality available in some other SQL dialects. Refer to the do The

[GitHub] [druid] clintropolis commented on a change in pull request #11188: SQL timeseries no longer skip empty buckets with all granularity

2021-05-07 Thread GitBox
clintropolis commented on a change in pull request #11188: URL: https://github.com/apache/druid/pull/11188#discussion_r628486125 ## File path: docs/querying/sql.md ## @@ -293,15 +293,15 @@ the `UNNEST` functionality available in some other SQL dialects. Refer to the do The

[GitHub] [druid] techdocsmith opened a new pull request #11218: change errant 'none' configuration to 'manual':

2021-05-07 Thread GitBox
techdocsmith opened a new pull request #11218: URL: https://github.com/apache/druid/pull/11218 Fixes incorrect reference to `none` instead of `manual` for `druid.query.scheduler.prioritization.strategy`. This PR has: - [X] been self-reviewed. -- This is an

[druid] branch master updated (a1f850d -> 3b9dad4)

2021-05-07 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/druid.git. from a1f850d Fix vectorized cardinality bug on certain string columns. (#11199) add 3b9dad4 Consolidate the

[GitHub] [druid] himanshug merged pull request #11187: Consolidate the number of Dockerfiles

2021-05-07 Thread GitBox
himanshug merged pull request #11187: URL: https://github.com/apache/druid/pull/11187 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service,

[GitHub] [druid] himanshug commented on pull request #11187: Consolidate the number of Dockerfiles

2021-05-07 Thread GitBox
himanshug commented on pull request #11187: URL: https://github.com/apache/druid/pull/11187#issuecomment-834622212 I restarted the build, looked like transient failure. But, good after build. Thanks -- This is an automated message from the Apache Git Service. To respond to the message,

[druid] branch master updated (a8c00d8 -> a1f850d)

2021-05-07 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/druid.git. from a8c00d8 chore: fix case of GitHub (#10928) add a1f850d Fix vectorized cardinality bug on certain string columns.

[GitHub] [druid] gianm merged pull request #11199: Fix vectorized cardinality bug on certain string columns.

2021-05-07 Thread GitBox
gianm merged pull request #11199: URL: https://github.com/apache/druid/pull/11199 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please

[GitHub] [druid] clintropolis commented on a change in pull request #11199: Fix vectorized cardinality bug on certain string columns.

2021-05-07 Thread GitBox
clintropolis commented on a change in pull request #11199: URL: https://github.com/apache/druid/pull/11199#discussion_r628061017 ## File path: processing/src/main/java/org/apache/druid/segment/VectorColumnProcessorFactory.java ## @@ -41,17 +41,25 @@ public interface

[GitHub] [druid] abhishekagarwal87 commented on a change in pull request #11188: SQL timeseries no longer skip empty buckets with all granularity

2021-05-07 Thread GitBox
abhishekagarwal87 commented on a change in pull request #11188: URL: https://github.com/apache/druid/pull/11188#discussion_r627293879 ## File path: docs/querying/sql.md ## @@ -293,15 +293,15 @@ the `UNNEST` functionality available in some other SQL dialects. Refer to the do

[druid] branch 0.21.1 updated: Upgrade jetty version (#11207) (#11216)

2021-05-07 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch 0.21.1 in repository https://gitbox.apache.org/repos/asf/druid.git The following commit(s) were added to refs/heads/0.21.1 by this push: new f588346 Upgrade jetty version (#11207) (#11216)

[GitHub] [druid] clintropolis merged pull request #11216: [Backport] Upgrade jetty version

2021-05-07 Thread GitBox
clintropolis merged pull request #11216: URL: https://github.com/apache/druid/pull/11216 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service,

[GitHub] [druid] clintropolis merged pull request #11217: [Backport] Suppressing false positive CVE-2020-7791

2021-05-07 Thread GitBox
clintropolis merged pull request #11217: URL: https://github.com/apache/druid/pull/11217 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service,

[druid] branch 0.21.1 updated: Suppressing false positive CVE-2020-7791 (#11215) (#11217)

2021-05-07 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch 0.21.1 in repository https://gitbox.apache.org/repos/asf/druid.git The following commit(s) were added to refs/heads/0.21.1 by this push: new db67938 Suppressing false positive

[GitHub] [druid] clintropolis commented on pull request #11217: [Backport] Suppressing false positive CVE-2020-7791

2021-05-07 Thread GitBox
clintropolis commented on pull request #11217: URL: https://github.com/apache/druid/pull/11217#issuecomment-834165827 travis failure seems unrelated since is also failing on #11216 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [druid] clintropolis commented on pull request #11216: [Backport] Upgrade jetty version

2021-05-07 Thread GitBox
clintropolis commented on pull request #11216: URL: https://github.com/apache/druid/pull/11216#issuecomment-834163891 the travis failure doesn't seem related to this PR, as I see it on #11217 as well -- This is an automated message from the Apache Git Service. To respond to the message,

[druid] branch master updated (49c8307 -> a8c00d8)

2021-05-07 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/druid.git. from 49c8307 Update datasource.md (#10864) add a8c00d8 chore: fix case of GitHub (#10928) No new revisions were

[GitHub] [druid] clintropolis merged pull request #10928: chore: fix case of GitHub

2021-05-07 Thread GitBox
clintropolis merged pull request #10928: URL: https://github.com/apache/druid/pull/10928 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service,

[druid] branch master updated (9be2a5c -> 49c8307)

2021-05-07 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/druid.git. from 9be2a5c Add documentation re alphabetical sorted of MV dimensions (#10695) add 49c8307 Update datasource.md

[GitHub] [druid] clintropolis merged pull request #10864: Update datasource.md

2021-05-07 Thread GitBox
clintropolis merged pull request #10864: URL: https://github.com/apache/druid/pull/10864 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service,

[druid] branch master updated (d73f72e -> 9be2a5c)

2021-05-07 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/druid.git. from d73f72e Add feature to automatically remove supervisor based on retention period (#11200) add 9be2a5c Add

[GitHub] [druid] clintropolis merged pull request #10695: Add documentation re alphabetical sorted of MV dimensions

2021-05-07 Thread GitBox
clintropolis merged pull request #10695: URL: https://github.com/apache/druid/pull/10695 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service,

[GitHub] [druid] lgtm-com[bot] commented on pull request #11208: fix count and average SQL aggregators on constant virtual columns

2021-05-07 Thread GitBox
lgtm-com[bot] commented on pull request #11208: URL: https://github.com/apache/druid/pull/11208#issuecomment-834093942 This pull request **introduces 1 alert** when merging 4ee69fbd7b43b980220554da291ea0db664f48ed into 351059ca435233a5ddf60ef998b2fd4f3612f1e2 - [view on