[GitHub] [incubator-druid] ccaominh commented on issue #8230: Fix dependency analyze warnings

2019-09-06 Thread GitBox
ccaominh commented on issue #8230: Fix dependency analyze warnings URL: https://github.com/apache/incubator-druid/pull/8230#issuecomment-529058928 Updated diff of distribution: ``` diff -r before3/LICENSE after5/LICENSE 331,333d330 < This product bundles Apache Commons CLI

[GitHub] [incubator-druid] ccaominh commented on issue #8230: Fix dependency analyze warnings

2019-09-06 Thread GitBox
ccaominh commented on issue #8230: Fix dependency analyze warnings URL: https://github.com/apache/incubator-druid/pull/8230#issuecomment-528976154 I've manually tested the latest commit with the following tutorials: - https://druid.apache.org/docs/latest/tutorials/tutorial-batch.html

[GitHub] [incubator-druid] ccaominh commented on issue #8230: Fix dependency analyze warnings

2019-09-05 Thread GitBox
ccaominh commented on issue #8230: Fix dependency analyze warnings URL: https://github.com/apache/incubator-druid/pull/8230#issuecomment-528660058 Updated diff after resolving merge conflicts (diff is same as https://github.com/apache/incubator-druid/pull/8230#issuecomment-528104856,

[GitHub] [incubator-druid] ccaominh commented on issue #8230: Fix dependency analyze warnings

2019-09-04 Thread GitBox
ccaominh commented on issue #8230: Fix dependency analyze warnings URL: https://github.com/apache/incubator-druid/pull/8230#issuecomment-528104856 Updated diff of the druid distribution before/after the changes in this PR: ``` diff -r before/LICENSE after2/LICENSE 331,333d330 <

[GitHub] [incubator-druid] ccaominh commented on issue #8230: Fix dependency analyze warnings

2019-08-27 Thread GitBox
ccaominh commented on issue #8230: Fix dependency analyze warnings URL: https://github.com/apache/incubator-druid/pull/8230#issuecomment-525493035 Diff of distribution before and after this PR (`diff -r apache-druid-0.16.0-incubating-SNAPSHOT-before

[GitHub] [incubator-druid] ccaominh commented on issue #8230: Fix dependency analyze warnings

2019-08-27 Thread GitBox
ccaominh commented on issue #8230: Fix dependency analyze warnings URL: https://github.com/apache/incubator-druid/pull/8230#issuecomment-525435918 @Dylan1312 This PR undoes some of the changes in #8339 by re-adding `hadoop-mapreduce-client-core` to `indexing-hadoop/pom.xml` to satisfy