Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2113#discussion_r181659113
--- Diff:
core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java ---
@@ -681,10 +681,10 @@ private static void
deletePhysicalPartition(List<PartitionSpec> partitionSpecs,
FileFactory.deleteAllCarbonFilesOfDir(FileFactory.getCarbonFile(location.toString()));
}
} else {
- // delete the segment folder if it is empty
+ // delete the segment folder
CarbonFile file = FileFactory.getCarbonFile(location.toString());
--- End diff --
change `file` to `segmentPath`
---
