[GitHub] leventov commented on a change in pull request #5980: Various changes about druid-services module

2018-07-10 Thread GitBox
leventov commented on a change in pull request #5980: Various changes about druid-services module URL: https://github.com/apache/incubator-druid/pull/5980#discussion_r201368627 ## File path: services/src/main/java/io/druid/cli/CliCoordinator.java ## @@ -255,6 +255,6 @@

[GitHub] leventov commented on a change in pull request #5980: Various changes about druid-services module

2018-07-10 Thread GitBox
leventov commented on a change in pull request #5980: Various changes about druid-services module URL: https://github.com/apache/incubator-druid/pull/5980#discussion_r201368904 ## File path: services/src/main/java/io/druid/cli/CliInternalHadoopIndexer.java ## @@ -150,7

[GitHub] leventov commented on a change in pull request #5980: Various changes about druid-services module

2018-07-10 Thread GitBox
leventov commented on a change in pull request #5980: Various changes about druid-services module URL: https://github.com/apache/incubator-druid/pull/5980#discussion_r201370593 ## File path: services/src/main/java/io/druid/cli/CliPeon.java ## @@ -243,7 +242,7 @@ public

[GitHub] leventov commented on a change in pull request #5980: Various changes about druid-services module

2018-07-10 Thread GitBox
leventov commented on a change in pull request #5980: Various changes about druid-services module URL: https://github.com/apache/incubator-druid/pull/5980#discussion_r201365909 ## File path:

[GitHub] leventov commented on a change in pull request #5980: Various changes about druid-services module

2018-07-10 Thread GitBox
leventov commented on a change in pull request #5980: Various changes about druid-services module URL: https://github.com/apache/incubator-druid/pull/5980#discussion_r201367693 ## File path: server/src/test/java/io/druid/initialization/InitializationTest.java ## @@ -454,8