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?
---
