You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@carbondata.apache.org by GitBox <gi...@apache.org> on 2019/03/21 13:27:08 UTC

[GitHub] [carbondata] akashrn5 commented on a change in pull request #3156: [CARBONDATA-3320]fix number of partitions issue in describe formatted and drop partition issue

akashrn5 commented on a change in pull request #3156: [CARBONDATA-3320]fix number of partitions issue in describe formatted and drop partition issue
URL: https://github.com/apache/carbondata/pull/3156#discussion_r267754656
 
 

 ##########
 File path: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java
 ##########
 @@ -861,6 +858,18 @@ private static void deletePhysicalPartition(List<PartitionSpec> partitionSpecs,
     }
   }
 
+  /**
+   * This method deletes the directories recursively if there are no files under that folder
 
 Review comment:
   here i have added function because, i need to traverse recursively and check and delete the parent folder if there are not files present under that folder.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services