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