[GitHub] jihoonson commented on a change in pull request #6068: Prohibit Lists.newArrayList() with a single argument

2018-07-30 Thread GitBox
jihoonson commented on a change in pull request #6068: Prohibit Lists.newArrayList() with a single argument URL: https://github.com/apache/incubator-druid/pull/6068#discussion_r206277090 ## File path: indexing-service/src/main/java/io/druid/indexing/common/task/Tasks.java

[GitHub] jihoonson commented on a change in pull request #6068: Prohibit Lists.newArrayList() with a single argument

2018-07-30 Thread GitBox
jihoonson commented on a change in pull request #6068: Prohibit Lists.newArrayList() with a single argument URL: https://github.com/apache/incubator-druid/pull/6068#discussion_r206274914 ## File path: processing/src/main/java/io/druid/collections/spatial/Node.java ## @@

[GitHub] jihoonson commented on a change in pull request #6068: Prohibit Lists.newArrayList() with a single argument

2018-07-30 Thread GitBox
jihoonson commented on a change in pull request #6068: Prohibit Lists.newArrayList() with a single argument URL: https://github.com/apache/incubator-druid/pull/6068#discussion_r206272902 ## File path: indexing-service/src/main/java/io/druid/indexing/common/task/Tasks.java

[GitHub] jihoonson commented on a change in pull request #6068: Prohibit Lists.newArrayList() with a single argument

2018-07-29 Thread GitBox
jihoonson commented on a change in pull request #6068: Prohibit Lists.newArrayList() with a single argument URL: https://github.com/apache/incubator-druid/pull/6068#discussion_r205986835 ## File path: processing/src/main/java/io/druid/collections/spatial/Node.java ## @@