This is an automated email from the ASF dual-hosted git repository.

jnioche pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/storm.git


The following commit(s) were added to refs/heads/master by this push:
     new 06fa5b9dc HdfsBlobStoreFile set wrong permission for file
     new 32cab5573 Merge pull request #3506 from skysiders/STORM-3888
06fa5b9dc is described below

commit 06fa5b9dc94f1cf971f4b8ea56e7fb1a11783335
Author: skysiders <[email protected]>
AuthorDate: Sat Dec 24 23:26:11 2022 +0800

    HdfsBlobStoreFile set wrong permission for file
---
 .../main/java/org/apache/storm/hdfs/blobstore/HdfsBlobStoreFile.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/external/storm-hdfs-blobstore/src/main/java/org/apache/storm/hdfs/blobstore/HdfsBlobStoreFile.java
 
b/external/storm-hdfs-blobstore/src/main/java/org/apache/storm/hdfs/blobstore/HdfsBlobStoreFile.java
index fe64f1757..f124cdfd0 100644
--- 
a/external/storm-hdfs-blobstore/src/main/java/org/apache/storm/hdfs/blobstore/HdfsBlobStoreFile.java
+++ 
b/external/storm-hdfs-blobstore/src/main/java/org/apache/storm/hdfs/blobstore/HdfsBlobStoreFile.java
@@ -147,7 +147,7 @@ public class HdfsBlobStoreFile extends BlobStoreFile {
                 fileSystem.setPermission(path.getParent(), dirperms);
             }
             out = fileSystem.create(path, (short) 
this.getMetadata().get_replication_factor());
-            fileSystem.setPermission(path, dirperms);
+            fileSystem.setPermission(path, fileperms);
             fileSystem.setReplication(path, (short) 
this.getMetadata().get_replication_factor());
         }
         if (out == null) {

Reply via email to