Github user manishgupta88 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2107#discussion_r177410099
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/store/CarbonFactDataHandlerModel.java
---
@@ -338,7 +339,23 @@ public static CarbonFactDataHandlerModel
getCarbonFactDataHandlerModel(CarbonLoa
*/
private static String
getCarbonDataFolderLocation(CarbonDataLoadConfiguration configuration) {
if (configuration.getDataWritePath() != null) {
- CarbonUtil.checkAndCreateFolder(configuration.getDataWritePath());
+ String paths = configuration.getDataWritePath();
+ AbsoluteTableIdentifier absoluteTableIdentifier =
configuration.getTableIdentifier();
+ String partPath = absoluteTableIdentifier.getTablePath();
+ String[] partDirs = paths.split(partPath);
+ // it will create folder one by one and apply the permissions
+ // else creation of folder in one go will set the permission for
last directory only
+ if (partDirs.length > 1) {
--- End diff --
Can you add an example above with paths or partDirs for better understanding
---
