[GitHub] [druid] FrankChen021 commented on issue #12372: Could not resolve type id 'variance' as a subtype of `org.apache.druid.query.aggregation.AggregatorFactory`

2022-03-28 Thread GitBox
FrankChen021 commented on issue #12372: URL: https://github.com/apache/druid/issues/12372#issuecomment-1081338120 Is `druid-stats` loaded on historical ? -- 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] FrankChen021 commented on issue #12374: java needs to be available in PATH for Druid to run properly.

2022-03-28 Thread GitBox
FrankChen021 commented on issue #12374: URL: https://github.com/apache/druid/issues/12374#issuecomment-1081332883 Druid finds JRE in following order: `DRUID_JAVA_HOME`, `JAVA_HOME`, `PATH`. The tutorial/index.md doc only mentions about the first two. > Druid relies on the

[GitHub] [druid] techdocsmith commented on a change in pull request #12371: Update ingestion-spec.md

2022-03-28 Thread GitBox
techdocsmith commented on a change in pull request #12371: URL: https://github.com/apache/druid/pull/12371#discussion_r836992003 ## File path: docs/ingestion/ingestion-spec.md ## @@ -205,7 +205,7 @@ A `dimensionsSpec` can have the following components: | Field

[GitHub] [druid] FrankChen021 commented on pull request #12370: Package kinesis client jar within the extension

2022-03-28 Thread GitBox
FrankChen021 commented on pull request #12370: URL: https://github.com/apache/druid/pull/12370#issuecomment-1081322939 license file should be updated. ``` Error: found 3 missing licenses. These licenses are reported, but missing in the registry druid_module:

[GitHub] [druid] FrankChen021 commented on a change in pull request #12373: Deprecated jackson fix

2022-03-28 Thread GitBox
FrankChen021 commented on a change in pull request #12373: URL: https://github.com/apache/druid/pull/12373#discussion_r836989594 ## File path: core/src/main/java/org/apache/druid/guice/GuiceAnnotationIntrospector.java ## @@ -58,9 +58,9 @@ public Object

[GitHub] [druid] clintropolis merged pull request #12365: Add an integration test for null-only columns

2022-03-28 Thread GitBox
clintropolis merged pull request #12365: URL: https://github.com/apache/druid/pull/12365 -- 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. To unsubscribe, e-mail:

[druid] branch master updated: Add an integration test for null-only columns (#12365)

2022-03-28 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/druid.git The following commit(s) were added to refs/heads/master by this push: new 49a3f42 Add an integration test for null-only

[druid] branch dependabot/maven/io.opentelemetry.instrumentation-opentelemetry-instrumentation-bom-alpha-1.12.1-alpha created (now 368c315)

2022-03-28 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/io.opentelemetry.instrumentation-opentelemetry-instrumentation-bom-alpha-1.12.1-alpha in repository https://gitbox.apache.org/repos/asf/druid.git. at 368c315 Bump

[GitHub] [druid] dependabot[bot] opened a new pull request #12375: Bump opentelemetry-instrumentation-bom-alpha from 1.7.0-alpha to 1.12.1-alpha

2022-03-28 Thread GitBox
dependabot[bot] opened a new pull request #12375: URL: https://github.com/apache/druid/pull/12375 Bumps [opentelemetry-instrumentation-bom-alpha](https://github.com/open-telemetry/opentelemetry-java-instrumentation) from 1.7.0-alpha to 1.12.1-alpha. Release notes Sourced from

[GitHub] [druid] AlanyTan opened a new issue #12374: java needs to be available in PATH for Druid to run properly.

2022-03-28 Thread GitBox
AlanyTan opened a new issue #12374: URL: https://github.com/apache/druid/issues/12374 tasks fails ### Affected Version 0.22.1 ### Description **- Cluster size:** 4 core 16GB RAM **- Configurations in use:** micro **- Steps to reproduce the problem:** after

[GitHub] [druid] jimjoamz opened a new pull request #12373: Deprecated jackson fix

2022-03-28 Thread GitBox
jimjoamz opened a new pull request #12373: URL: https://github.com/apache/druid/pull/12373 ### Description Druid uses deprecated Jackson methods `getGenericType`. This method was deprecated in Jackson 2.7 and is dropped in Jackson 2.11 This PR allows any version of Druid

[druid] branch master updated (f2495a6 -> 9ed7aa3)

2022-03-28 Thread techdocsmith
This is an automated email from the ASF dual-hosted git repository. techdocsmith pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/druid.git. from f2495a6 fix messageGap metric (#12337) add 9ed7aa3 Docs for request logging (#12363) No new revisions

[GitHub] [druid] clintropolis commented on a change in pull request #12345: Introducing a new config to ignore nulls while computing String Cardinality

2022-03-28 Thread GitBox
clintropolis commented on a change in pull request #12345: URL: https://github.com/apache/druid/pull/12345#discussion_r836874680 ## File path: core/src/main/java/org/apache/druid/common/config/NullValueHandlingConfig.java ## @@ -26,20 +26,46 @@ { public static final

[GitHub] [druid] AbdelouahabKhelifati opened a new issue #12372: Could not resolve type id 'variance' as a subtype of `org.apache.druid.query.aggregation.AggregatorFactory`

2022-03-28 Thread GitBox
AbdelouahabKhelifati opened a new issue #12372: URL: https://github.com/apache/druid/issues/12372 Please provide a detailed title (e.g. "Broker crashes when using TopN query with Bound filter" instead of just "Broker crashes"). ### Affected Version Druid 0.22.0 ###

[GitHub] [druid] techdocsmith merged pull request #12363: Docs for request logging

2022-03-28 Thread GitBox
techdocsmith merged pull request #12363: URL: https://github.com/apache/druid/pull/12363 -- 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. To unsubscribe, e-mail:

[GitHub] [druid] vtlim commented on a change in pull request #12363: Docs for request logging

2022-03-28 Thread GitBox
vtlim commented on a change in pull request #12363: URL: https://github.com/apache/druid/pull/12363#discussion_r836805094 ## File path: docs/configuration/index.md ## @@ -279,24 +280,24 @@ For SQL query request, the `native_query` field is empty. Example

[GitHub] [druid] vtlim commented on a change in pull request #12363: Docs for request logging

2022-03-28 Thread GitBox
vtlim commented on a change in pull request #12363: URL: https://github.com/apache/druid/pull/12363#discussion_r836805094 ## File path: docs/configuration/index.md ## @@ -279,24 +280,24 @@ For SQL query request, the `native_query` field is empty. Example

[GitHub] [druid] vtlim commented on a change in pull request #12363: Docs for request logging

2022-03-28 Thread GitBox
vtlim commented on a change in pull request #12363: URL: https://github.com/apache/druid/pull/12363#discussion_r836801217 ## File path: docs/operations/metrics.md ## @@ -72,11 +69,11 @@ Available Metrics |`segment/scan/pending`|Number of segments in queue waiting to be

[GitHub] [druid] vtlim commented on a change in pull request #12363: Docs for request logging

2022-03-28 Thread GitBox
vtlim commented on a change in pull request #12363: URL: https://github.com/apache/druid/pull/12363#discussion_r836800323 ## File path: docs/configuration/index.md ## @@ -310,31 +311,36 @@ For native query, the following MDC fields are populated with `setMDC`:

[GitHub] [druid] vtlim commented on a change in pull request #12363: Docs for request logging

2022-03-28 Thread GitBox
vtlim commented on a change in pull request #12363: URL: https://github.com/apache/druid/pull/12363#discussion_r836799537 ## File path: docs/configuration/index.md ## @@ -310,31 +311,36 @@ For native query, the following MDC fields are populated with `setMDC`:

[GitHub] [druid] techdocsmith commented on a change in pull request #12363: Docs for request logging

2022-03-28 Thread GitBox
techdocsmith commented on a change in pull request #12363: URL: https://github.com/apache/druid/pull/12363#discussion_r836763347 ## File path: docs/configuration/index.md ## @@ -279,24 +280,24 @@ For SQL query request, the `native_query` field is empty. Example

[GitHub] [druid] mark-imply opened a new pull request #12371: Update ingestion-spec.md

2022-03-28 Thread GitBox
mark-imply opened a new pull request #12371: URL: https://github.com/apache/druid/pull/12371 ### Description Added best practice point to dimensions description. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [druid] paul-rogers commented on pull request #12368: First cut at restructuring the integration tests

2022-03-28 Thread GitBox
paul-rogers commented on pull request #12368: URL: https://github.com/apache/druid/pull/12368#issuecomment-1080945084 This is a big PR. It is all-in-one so folks can see the whole picture. If it helps, this can be broken into smaller chunks, at loss of overall context. Here's a

[GitHub] [druid] paul-rogers commented on issue #12359: Simplify and Accelerate Integration Tests

2022-03-28 Thread GitBox
paul-rogers commented on issue #12359: URL: https://github.com/apache/druid/issues/12359#issuecomment-1080857616 There is a PR for this work: #12368. With this, it is now possible to: 1. Build Druid in the normal way. 2. Build the test image: about 1 minute. 3. Run a test:

[GitHub] [druid] a2l007 merged pull request #12337: fix messageGap metric

2022-03-28 Thread GitBox
a2l007 merged pull request #12337: URL: https://github.com/apache/druid/pull/12337 -- 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. To unsubscribe, e-mail:

[druid] branch master updated: fix messageGap metric (#12337)

2022-03-28 Thread atul
This is an automated email from the ASF dual-hosted git repository. atul pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/druid.git The following commit(s) were added to refs/heads/master by this push: new f2495a6 fix messageGap metric (#12337) f2495a6 is

[GitHub] [druid] AmatyaAvadhanula opened a new pull request #12370: Package kinesis client jar within the extension

2022-03-28 Thread GitBox
AmatyaAvadhanula opened a new pull request #12370: URL: https://github.com/apache/druid/pull/12370 ### Description amazon-kinesis-client was not covered undered the apache license and required separate insertion in the kinesis extension. This can now be avoided

[druid] branch master updated: Use javaOptsArray provided in task context (#12326)

2022-03-28 Thread kfaraz
This is an automated email from the ASF dual-hosted git repository. kfaraz pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/druid.git The following commit(s) were added to refs/heads/master by this push: new 9c6b9ab Use javaOptsArray provided in task

[GitHub] [druid] kfaraz merged pull request #12326: Utilize javaOptsArray provided as part of task context

2022-03-28 Thread GitBox
kfaraz merged pull request #12326: URL: https://github.com/apache/druid/pull/12326 -- 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. To unsubscribe, e-mail:

[GitHub] [druid] lgtm-com[bot] commented on pull request #12326: Utilize javaOptsArray provided as part of task context

2022-03-28 Thread GitBox
lgtm-com[bot] commented on pull request #12326: URL: https://github.com/apache/druid/pull/12326#issuecomment-1080493667 This pull request **introduces 1 alert** when merging b2207dcae09d045d17283d3165beb0beb9a7f48e into ee44fe45c66c10af0746bf053faed2a61df6168b - [view on