Github user shardul-cr7 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2739#discussion_r219739583 --- Diff: core/src/main/java/org/apache/carbondata/core/util/CarbonUtil.java --- @@ -2226,7 +2226,11 @@ public static String getFilePathExternalFilePath(String path, Configuration conf if (dataFile.getName().endsWith(CarbonCommonConstants.FACT_FILE_EXT)) { return dataFile.getAbsolutePath(); } else if (dataFile.isDirectory()) { - return getFilePathExternalFilePath(dataFile.getAbsolutePath(), configuration); + if (getFilePathExternalFilePath(dataFile.getAbsolutePath(), configuration) == null) { --- End diff -- Handled the review comment and added the test cases also for the scenarios.
---