[GitHub] [druid] clintropolis commented on issue #9736: HLLSketchMerge aggregator failing for some metrics after upgrade to v0.18

2020-04-22 Thread GitBox
clintropolis commented on issue #9736: URL: https://github.com/apache/druid/issues/9736#issuecomment-618197919 I ran some tests, and it appears that downgrading `datasketches-java` to `1.1.0-incubating` doesn't have this issue. Since we have an unrelated major regression with stream ingest

[GitHub] [druid] clintropolis opened a new pull request #9751: revert datasketches-java version to 1.1.0-incubating until new version is released

2020-04-22 Thread GitBox
clintropolis opened a new pull request #9751: URL: https://github.com/apache/druid/pull/9751 Temporarily switch `datasketches-java` back to `1.1.0-incubating` in case the new version with the fix for #9736 doesn't make it in on the timescale we release `0.18.1`. -

[GitHub] [druid] clintropolis edited a comment on issue #9736: HLLSketchMerge aggregator failing for some metrics after upgrade to v0.18

2020-04-22 Thread GitBox
clintropolis edited a comment on issue #9736: URL: https://github.com/apache/druid/issues/9736#issuecomment-618189702 > Is there a schema file somewhere that describes the layout of the hll_segment.zip file that @scrawfor attached to this issue. I'd like to write a parser so that in the fu

[GitHub] [druid] clintropolis commented on issue #9736: HLLSketchMerge aggregator failing for some metrics after upgrade to v0.18

2020-04-22 Thread GitBox
clintropolis commented on issue #9736: URL: https://github.com/apache/druid/issues/9736#issuecomment-618189702 > Is there a schema file somewhere that describes the layout of the hll_segment.zip file that @scrawfor attached to this issue. I'd like to write a parser so that in the future if

[GitHub] [druid] leerho commented on issue #9736: HLLSketchMerge aggregator failing for some metrics after upgrade to v0.18

2020-04-22 Thread GitBox
leerho commented on issue #9736: URL: https://github.com/apache/druid/issues/9736#issuecomment-618176172 Is there a schema file somewhere that describes the layout of the hll_segment.zip file that @scrawfor attached to this issue. I'd like to write a parser so that in the future if we nee

[GitHub] [druid] stale[bot] commented on issue #8097: Custom Json extractors for Kafka lookup module

2020-04-22 Thread GitBox
stale[bot] commented on issue #8097: URL: https://github.com/apache/druid/issues/8097#issuecomment-618163919 This issue has been marked as stale due to 280 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If this issue is still relevant, please simply write a

[GitHub] [druid] stale[bot] commented on issue #9057: [new feature] Add HBase2 Indexer

2020-04-22 Thread GitBox
stale[bot] commented on issue #9057: URL: https://github.com/apache/druid/pull/9057#issuecomment-618163901 This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If you think that's incorrect or this pull request

[GitHub] [druid] stale[bot] commented on issue #6199: [Proposal] Vectorized querying for lookups

2020-04-22 Thread GitBox
stale[bot] commented on issue #6199: URL: https://github.com/apache/druid/issues/6199#issuecomment-618163925 This issue has been marked as stale due to 280 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If this issue is still relevant, please simply write a

[GitHub] [druid] licl2014 commented on a change in pull request #5001: Fix removeDataSource() and removeSegment() of SQLMetadataSegmentManager

2020-04-22 Thread GitBox
licl2014 commented on a change in pull request #5001: URL: https://github.com/apache/druid/pull/5001#discussion_r413489633 ## File path: server/src/main/java/io/druid/metadata/SQLMetadataSegmentManager.java ## @@ -144,26 +142,35 @@ public void run() delay.getMillis()

[GitHub] [druid] leerho commented on issue #9736: HLLSketchMerge aggregator failing for some metrics after upgrade to v0.18

2020-04-22 Thread GitBox
leerho commented on issue #9736: URL: https://github.com/apache/druid/issues/9736#issuecomment-618145618 We have a fix! @clintropolis @scrawfor @gianm @AlexanderSaydakov I want to thank all of you for your help! This was truly a team effort! With clues from @clintropol

[GitHub] [druid] jinleileiking opened a new issue #9750: docker hub should have a mysql connector image

2020-04-22 Thread GitBox
jinleileiking opened a new issue #9750: URL: https://github.com/apache/druid/issues/9750 ``` This image contains solely the postgres metadata storage connector. If you need the mysql metadata storage connector, you can use Dockerfile.mysql to add it to the base image above. ```

[GitHub] [druid] mooseen commented on issue #8438: druid-0.15.0运行后自动进程退出

2020-04-22 Thread GitBox
mooseen commented on issue #8438: URL: https://github.com/apache/druid/issues/8438#issuecomment-618136707 I had encountered the same issue with druid version 0.17.1. All process of druid cluster will shutdown at an indefinite time in the future and this include supervise process too.

[GitHub] [druid] viongpanzi commented on issue #9722: Optimize FileWriteOutBytes to avoid high system cpu usage

2020-04-22 Thread GitBox
viongpanzi commented on issue #9722: URL: https://github.com/apache/druid/pull/9722#issuecomment-618132922 > @viongpanzi - The intelliJ inspection errors still exist in this patch. You can run intelliJ inspections locally from the editor like this. > https://user-images.githubusercontent

[GitHub] [druid] ccaominh commented on issue #8987: Adding support for autoscaling in GCE

2020-04-22 Thread GitBox
ccaominh commented on issue #8987: URL: https://github.com/apache/druid/pull/8987#issuecomment-618120363 > > > Also, what do you think about adding an integration test for the new extension? I think having an integration test will help ensure future contributions do not break functionality

[GitHub] [druid] clintropolis commented on issue #8987: Adding support for autoscaling in GCE

2020-04-22 Thread GitBox
clintropolis commented on issue #8987: URL: https://github.com/apache/druid/pull/8987#issuecomment-618111432 > >Also, what do you think about adding an integration test for the new extension? I think having an integration test will help ensure future contributions do not break functionalit

[GitHub] [druid] clintropolis commented on issue #9736: HLLSketchMerge aggregator failing for some metrics after upgrade to v0.18

2020-04-22 Thread GitBox
clintropolis commented on issue #9736: URL: https://github.com/apache/druid/issues/9736#issuecomment-618105737 The exception occurs [on this line on](https://github.com/apache/druid/blob/master/extensions-core/datasketches/src/main/java/org/apache/druid/query/aggregation/datasketches/hll/Hl

[GitHub] [druid] scrawfor commented on issue #9736: HLLSketchMerge aggregator failing for some metrics after upgrade to v0.18

2020-04-22 Thread GitBox
scrawfor commented on issue #9736: URL: https://github.com/apache/druid/issues/9736#issuecomment-618092143 0.16.0 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

[GitHub] [druid] AlexanderSaydakov commented on issue #9736: HLLSketchMerge aggregator failing for some metrics after upgrade to v0.18

2020-04-22 Thread GitBox
AlexanderSaydakov commented on issue #9736: URL: https://github.com/apache/druid/issues/9736#issuecomment-618084236 using what version of Druid? This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [druid] scrawfor commented on issue #9736: HLLSketchMerge aggregator failing for some metrics after upgrade to v0.18

2020-04-22 Thread GitBox
scrawfor commented on issue #9736: URL: https://github.com/apache/druid/issues/9736#issuecomment-618076075 @AlexanderSaydakov Using native batch indexer, local Firehose, built the sketch over a string column. Here's the metric spec. ```json { "type": "HLLSketchBui

[GitHub] [druid] scrawfor edited a comment on issue #9736: HLLSketchMerge aggregator failing for some metrics after upgrade to v0.18

2020-04-22 Thread GitBox
scrawfor edited a comment on issue #9736: URL: https://github.com/apache/druid/issues/9736#issuecomment-618076075 @AlexanderSaydakov Using native batch indexer, local Firehose, built the sketch over a string column. Here's the metric spec. ```json { "type": "HLLSk

[GitHub] [druid] AlexanderSaydakov commented on issue #9736: HLLSketchMerge aggregator failing for some metrics after upgrade to v0.18

2020-04-22 Thread GitBox
AlexanderSaydakov commented on issue #9736: URL: https://github.com/apache/druid/issues/9736#issuecomment-618072057 How was that segment with sketches created? This is an automated message from the Apache Git Service. To resp

[GitHub] [druid] scrawfor commented on issue #9736: HLLSketchMerge aggregator failing for some metrics after upgrade to v0.18

2020-04-22 Thread GitBox
scrawfor commented on issue #9736: URL: https://github.com/apache/druid/issues/9736#issuecomment-618065727 @clintropolis Thanks for confirming. I did find that the `HllSketchMergeAggregator` seemed to be affected as well. I tried to reissue the query this evening, but it's using `H

[GitHub] [druid] BigDataArtist opened a new issue #9749: Datasource still queries the data even after removing data from segments-cache?

2020-04-22 Thread GitBox
BigDataArtist opened a new issue #9749: URL: https://github.com/apache/druid/issues/9749 My Datasource in druid still reads the data from segments-cache( Which is historical data I guess) after removing the files cd var/druid/segment-cache/tableName/015-09-12T00:00:00.000Z_2015-

[GitHub] [druid] clintropolis commented on issue #9736: HLLSketchMerge aggregator failing for some metrics after upgrade to v0.18

2020-04-22 Thread GitBox
clintropolis commented on issue #9736: URL: https://github.com/apache/druid/issues/9736#issuecomment-618054731 Thanks for the segment and example query, I can reproduce this issue in the debugger with group by query and can confirm that it doesn't seem to affect timeseries queries, which m

[GitHub] [druid] suneet-s commented on issue #9722: Optimize FileWriteOutBytes to avoid high system cpu usage

2020-04-22 Thread GitBox
suneet-s commented on issue #9722: URL: https://github.com/apache/druid/pull/9722#issuecomment-618039402 @viongpanzi - The intelliJ inspection errors still exist in this patch. You can run intelliJ inspections locally from the editor like this. https://user-images.githubusercontent.com/4

[GitHub] [druid] suneet-s opened a new pull request #9748: Fix intelliJ inspection errors on top of PR 9722

2020-04-22 Thread GitBox
suneet-s opened a new pull request #9748: URL: https://github.com/apache/druid/pull/9748 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 g

[GitHub] [druid] clintropolis opened a new pull request #9747: [Backport] Fix misuse of Integer.SIZE in FileWriteOutBytes.writeInt

2020-04-22 Thread GitBox
clintropolis opened a new pull request #9747: URL: https://github.com/apache/druid/pull/9747 Backport of #9723 to 0.18.1. This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [druid] clintropolis opened a new pull request #9746: [Backport] fixes for inline subqueries when multi-value dimension is present

2020-04-22 Thread GitBox
clintropolis opened a new pull request #9746: URL: https://github.com/apache/druid/pull/9746 Backport of #9698 to 0.18.1. This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [druid] maytasm commented on issue #9745: [Backport] Integration tests for Kafka & Kinesis ingestion

2020-04-22 Thread GitBox
maytasm commented on issue #9745: URL: https://github.com/apache/druid/pull/9745#issuecomment-618015906 There were three merge conflicts which were simple to resolve. - integration-tests/pom.xml - integration-tests/src/test/java/org/apache/druid/tests/coordinator/duty/ITAutoCompaction

[GitHub] [druid] maytasm edited a comment on issue #9745: [Backport] Integration tests for Kafka & Kinesis ingestion

2020-04-22 Thread GitBox
maytasm edited a comment on issue #9745: URL: https://github.com/apache/druid/pull/9745#issuecomment-618014706 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[GitHub] [druid] maytasm commented on issue #9745: [Backport] Integration tests for Kafka & Kinesis ingestion

2020-04-22 Thread GitBox
maytasm commented on issue #9745: URL: https://github.com/apache/druid/pull/9745#issuecomment-618014706 This PR only modify files in integration-tests with the exception of: - .travis.yml : to add the new integration tests to our Travis CI pipeline - docs/development/extensions-core/ki

[GitHub] [druid] maytasm opened a new pull request #9745: [Backport] Integration tests for Kafka & Kinesis ingestion

2020-04-22 Thread GitBox
maytasm opened a new pull request #9745: URL: https://github.com/apache/druid/pull/9745 Backport of https://github.com/apache/druid/pull/9576 to 0.18.1. Backport of https://github.com/apache/druid/pull/9724 to 0.18.1. This

[GitHub] [druid] dtheodor commented on issue #9743: Crash when initializing the partial segment merge phase of index_parallel

2020-04-22 Thread GitBox
dtheodor commented on issue #9743: URL: https://github.com/apache/druid/issues/9743#issuecomment-617954300 Thanks. I'm avoiding this bug for now by targeting a single day interval in every task. I'm stuck with `org.apache.druid.java.util.common.IAE: Asked to add buffers[2,891,159,858] larg

[GitHub] [druid] singqwe opened a new issue #9744: Getting BufferOverflow exception when deserializing base64 serialized data

2020-04-22 Thread GitBox
singqwe opened a new issue #9744: URL: https://github.com/apache/druid/issues/9744 I am trying to test my custom aggregator using the druid binaries from quickstart tutorial which writes data in base64 using the [writeBinary](https://fasterxml.github.io/jackson-core/javadoc/2.2.0/com/faste

[GitHub] [druid] jihoonson commented on issue #9727: Markdown formatting error in druid-pac4j

2020-04-22 Thread GitBox
jihoonson commented on issue #9727: URL: https://github.com/apache/druid/pull/9727#issuecomment-617940105 @cwadding I am sorry, but reverted this PR since it was merged to 0.18.0. Would you please make a new PR that has the same fix for master?

[druid] branch 0.18.1 created (now 4230e3f)

2020-04-22 Thread jihoonson
This is an automated email from the ASF dual-hosted git repository. jihoonson pushed a change to branch 0.18.1 in repository https://gitbox.apache.org/repos/asf/druid.git. at 4230e3f Revert "Markdown formatting error in druid-pac4j (#9727)" No new revisions were added by this update. -

[druid] branch 0.18.0 updated: Revert "Markdown formatting error in druid-pac4j (#9727)"

2020-04-22 Thread jihoonson
This is an automated email from the ASF dual-hosted git repository. jihoonson pushed a commit to branch 0.18.0 in repository https://gitbox.apache.org/repos/asf/druid.git The following commit(s) were added to refs/heads/0.18.0 by this push: new 4230e3f Revert "Markdown formatting error in

[GitHub] [druid] BigDataArtist commented on issue #6919: how to load data from s3 if i dont deploy a cluster

2020-04-22 Thread GitBox
BigDataArtist commented on issue #6919: URL: https://github.com/apache/druid/issues/6919#issuecomment-617936482 Hi, My ingestion task says success when I import data from S3 to druid but then the druid never makes its data source , I am submitting the spec thru command line. #Comman

[druid] branch master updated: Add integration tests for kafka ingestion (#9724)

2020-04-22 Thread jihoonson
This is an automated email from the ASF dual-hosted git repository. jihoonson 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 16f5ae4 Add integration tests for kafka inges

[GitHub] [druid] jihoonson commented on issue #9724: Add integration tests for kafka ingestion

2020-04-22 Thread GitBox
jihoonson commented on issue #9724: URL: https://github.com/apache/druid/pull/9724#issuecomment-617917170 Travis failure was because of a timed out job. I restarted it. BTW, I think it would be nice to backport this PR into 0.18.1 since it increases our test coverage so that we can skip ma

[GitHub] [druid] ccaominh commented on issue #9743: Crash when initializing the partial segment merge phase of index_parallel

2020-04-22 Thread GitBox
ccaominh commented on issue #9743: URL: https://github.com/apache/druid/issues/9743#issuecomment-617915452 The `IndexOutOfBounds` exception during the `partial segment merge` phase of native batch ingestion is fixed by https://github.com/apache/druid/pull/9448 and is present in 0.18.0. -

[GitHub] [druid] jihoonson commented on a change in pull request #9724: Add integration tests for kafka ingestion

2020-04-22 Thread GitBox
jihoonson commented on a change in pull request #9724: URL: https://github.com/apache/druid/pull/9724#discussion_r413166935 ## File path: integration-tests/src/test/java/org/apache/druid/tests/coordinator/duty/ITAutoCompactionTest.java ## @@ -251,6 +251,9 @@ private void submi

[GitHub] [druid] viongpanzi commented on issue #9722: Optimize FileWriteOutBytes to avoid high system cpu usage

2020-04-22 Thread GitBox
viongpanzi commented on issue #9722: URL: https://github.com/apache/druid/pull/9722#issuecomment-617889235 > @viongpanzi Looks like the intelliJ inspections failure is valid (I now understand why you left that comment in the first patch. > > The correct fix here is to change the abst

[GitHub] [druid] gianm commented on issue #9707: Add IS_MULTIVALUE and COMPLEX_TYPE columns to INFORMATION_SCHEMA.COLUMNS

2020-04-22 Thread GitBox
gianm commented on issue #9707: URL: https://github.com/apache/druid/issues/9707#issuecomment-617887339 +1 on the concept. This would be a great addition. The DruidSchema is actually in a place already where it could know which columns are multivalue —  it gets it from SegmentMetadata queri

[GitHub] [druid] gianm commented on issue #9715: Druid Batch Load is Very slow

2020-04-22 Thread GitBox
gianm commented on issue #9715: URL: https://github.com/apache/druid/issues/9715#issuecomment-617885427 If you are doing batch loads, try making sure that there are enough parallel tasks running, and that your files are split up enough that we can launch enough parallel tasks. --

[GitHub] [druid] gianm commented on issue #9719: SQL query with Like can't add anything else

2020-04-22 Thread GitBox
gianm commented on issue #9719: URL: https://github.com/apache/druid/issues/9719#issuecomment-617884971 I just tried this on Druid 0.18.0 and it works OK. Perhaps it was fixed since 0.16. This is an automated message from th

[druid] branch master updated: Add QueryResource to log4j2 template. (#9735)

2020-04-22 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/druid.git The following commit(s) were added to refs/heads/master by this push: new 479c290 Add QueryResource to log4j2 template. (#97

[GitHub] [druid] scrawfor commented on issue #9736: HLLSketchMerge aggregator failing for some metrics after upgrade to v0.18

2020-04-22 Thread GitBox
scrawfor commented on issue #9736: URL: https://github.com/apache/druid/issues/9736#issuecomment-617857115 > @scrawfor Thanks for the upload. Do you have an example of a query that exhibits the problem? Sure. ```json { "dataSource" : "hlltest", "queryType" : "

[GitHub] [druid] suneet-s commented on issue #9722: Optimize FileWriteOutBytes to avoid high system cpu usage

2020-04-22 Thread GitBox
suneet-s commented on issue #9722: URL: https://github.com/apache/druid/pull/9722#issuecomment-61787 @viongpanzi Looks like the intelliJ inspections failure is valid (I now understand why you left that comment in the first patch. The correct fix here is to change the abstract met

[GitHub] [druid] gianm commented on issue #9736: HLLSketchMerge aggregator failing for some metrics after upgrade to v0.18

2020-04-22 Thread GitBox
gianm commented on issue #9736: URL: https://github.com/apache/druid/issues/9736#issuecomment-617836743 @scrawfor Thanks for the upload. Do you have an example of a query that exhibits the problem? This is an automated messa

[GitHub] [druid] scrawfor commented on issue #9736: HLLSketchMerge aggregator failing for some metrics after upgrade to v0.18

2020-04-22 Thread GitBox
scrawfor commented on issue #9736: URL: https://github.com/apache/druid/issues/9736#issuecomment-617801282 @gianm I pulled out all sensitive info and re-indexed our data, so I'm attaching the full segment. The offending metric is `unique_views_hll`. [hll_segment.zip](https://github.com

[GitHub] [druid] dtheodor opened a new issue #9743: Crash when initializing the partial segment merge phase of index_parallel

2020-04-22 Thread GitBox
dtheodor opened a new issue #9743: URL: https://github.com/apache/druid/issues/9743 I'm running druid 0.17.0, a parallel re-index from a druid data source with `single_dim` partioning and 12 concurrent tasks. There are two middle managers running, each in a node of 8 cores and 32GB ram. Th

[GitHub] [druid] frnidito commented on issue #8987: Adding support for autoscaling in GCE

2020-04-22 Thread GitBox
frnidito commented on issue #8987: URL: https://github.com/apache/druid/pull/8987#issuecomment-617605330 > For the integration test, we do have a group of integration tests that require the user to provide their own cloud and thus are not run by CI for this repo: https://github.com/apache

[GitHub] [druid] maytasm commented on a change in pull request #9724: Add integration tests for kafka ingestion

2020-04-22 Thread GitBox
maytasm commented on a change in pull request #9724: URL: https://github.com/apache/druid/pull/9724#discussion_r412733624 ## File path: integration-tests/src/test/java/org/apache/druid/tests/parallelized/ITKafkaIndexingServiceNonTransactionalParallelizedTest.java ## @@ -0,0 +1

[GitHub] [druid] maytasm commented on a change in pull request #9724: Add integration tests for kafka ingestion

2020-04-22 Thread GitBox
maytasm commented on a change in pull request #9724: URL: https://github.com/apache/druid/pull/9724#discussion_r412732882 ## File path: integration-tests/src/test/java/org/apache/druid/tests/indexer/AbstractStreamIndexingTest.java ## @@ -0,0 +1,439 @@ +/* + * Licensed to the A

[GitHub] [druid] maytasm commented on a change in pull request #9724: Add integration tests for kafka ingestion

2020-04-22 Thread GitBox
maytasm commented on a change in pull request #9724: URL: https://github.com/apache/druid/pull/9724#discussion_r412732569 ## File path: integration-tests/src/test/java/org/apache/druid/tests/indexer/AbstractStreamIndexingTest.java ## @@ -0,0 +1,439 @@ +/* + * Licensed to the A

[GitHub] [druid] maytasm commented on a change in pull request #9724: Add integration tests for kafka ingestion

2020-04-22 Thread GitBox
maytasm commented on a change in pull request #9724: URL: https://github.com/apache/druid/pull/9724#discussion_r412727283 ## File path: integration-tests/src/test/java/org/apache/druid/tests/indexer/AbstractStreamIndexingTest.java ## @@ -0,0 +1,439 @@ +/* + * Licensed to the A

[GitHub] [druid] maytasm commented on a change in pull request #9724: Add integration tests for kafka ingestion

2020-04-22 Thread GitBox
maytasm commented on a change in pull request #9724: URL: https://github.com/apache/druid/pull/9724#discussion_r412726527 ## File path: integration-tests/src/test/java/org/apache/druid/tests/indexer/AbstractKafkaIndexingServiceTest.java ## @@ -0,0 +1,156 @@ +/* + * Licensed to

[GitHub] [druid] maytasm commented on a change in pull request #9724: Add integration tests for kafka ingestion

2020-04-22 Thread GitBox
maytasm commented on a change in pull request #9724: URL: https://github.com/apache/druid/pull/9724#discussion_r412726606 ## File path: integration-tests/src/test/java/org/apache/druid/tests/indexer/AbstractIndexerTest.java ## @@ -57,7 +57,7 @@ protected TestQueryHelper que

[GitHub] [druid] frnidito commented on a change in pull request #8987: Adding support for autoscaling in GCE

2020-04-22 Thread GitBox
frnidito commented on a change in pull request #8987: URL: https://github.com/apache/druid/pull/8987#discussion_r412726317 ## File path: integration-tests/src/test/java/org/apache/druid/tests/coordinator/duty/ITAutoCompactionTest.java ## @@ -237,7 +237,7 @@ private void verify

[GitHub] [druid] maytasm commented on a change in pull request #9724: Add integration tests for kafka ingestion

2020-04-22 Thread GitBox
maytasm commented on a change in pull request #9724: URL: https://github.com/apache/druid/pull/9724#discussion_r412721348 ## File path: integration-tests/src/test/java/org/apache/druid/tests/indexer/AbstractIndexerTest.java ## @@ -57,7 +57,7 @@ protected TestQueryHelper que

[GitHub] [druid] maytasm commented on a change in pull request #9724: Add integration tests for kafka ingestion

2020-04-22 Thread GitBox
maytasm commented on a change in pull request #9724: URL: https://github.com/apache/druid/pull/9724#discussion_r412720994 ## File path: integration-tests/src/test/java/org/apache/druid/tests/coordinator/duty/ITAutoCompactionTest.java ## @@ -251,6 +251,9 @@ private void submitC