Github user ilooner commented on a diff in the pull request: https://github.com/apache/drill/pull/970#discussion_r145764035 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/FileSelection.java --- @@ -208,17 +208,17 @@ public boolean wasAllPartitionsPruned() { return this.wasAllPartitionsPruned; } - private static String commonPath(final List<FileStatus> statuses) { - if (statuses == null || statuses.isEmpty()) { - return ""; - } - - final List<String> files = Lists.newArrayList(); - for (final FileStatus status : statuses) { - files.add(status.getPath().toString()); - } - return commonPathForFiles(files); - } +// private static String commonPath(final List<FileStatus> statuses) { --- End diff -- Could we delete this instead of comment it out?
---