[GitHub] [hudi] xiarixiaoyao commented on a change in pull request #3183: [HUDI-2090]when hudi metadata is enabled, use different users to quer…

[GitBox <gi...@apache.org>]

xiarixiaoyao commented on a change in pull request #3183:
URL: https://github.com/apache/hudi/pull/3183#discussion_r661064156



##########
File path: hudi-common/src/main/java/org/apache/hudi/common/util/collection/DiskBasedMap.java
##########
@@ -111,8 +111,14 @@ private void initFile(File writeOnlyFile) throws IOException {
     if (writeOnlyFile.exists()) {
       writeOnlyFile.delete();
     }
-    if (!writeOnlyFile.getParentFile().exists()) {
-      writeOnlyFile.getParentFile().mkdir();
+    File parentDir = writeOnlyFile.getParentFile();
+    if (!parentDir.exists()) {
+      parentDir.mkdir();
+      // we should set all right for other user.

Review comment:
       ok， i will try to add UT。 thanks




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

To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org