You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by go...@apache.org on 2019/01/22 18:32:56 UTC

[beam] branch master updated: Set read permissions for the staged files

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

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


The following commit(s) were added to refs/heads/master by this push:
     new e8a2a11  Set read permissions for the staged files
     new 67e58e5  Merge pull request #7573 from angoenka/stage_permissions
e8a2a11 is described below

commit e8a2a1130766f04eec5f3ef840ad296e2a9d6af3
Author: Ankur <an...@users.noreply.github.com>
AuthorDate: Fri Jan 18 15:49:59 2019 -0800

    Set read permissions for the staged files
    
    Read permission are necessary for non root user.
---
 sdks/python/apache_beam/runners/portability/portable_stager.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/sdks/python/apache_beam/runners/portability/portable_stager.py b/sdks/python/apache_beam/runners/portability/portable_stager.py
index d4ca7a2..09ff18f 100644
--- a/sdks/python/apache_beam/runners/portability/portable_stager.py
+++ b/sdks/python/apache_beam/runners/portability/portable_stager.py
@@ -71,7 +71,8 @@ class PortableStager(Stager):
     def artifact_request_generator():
       artifact_metadata = beam_artifact_api_pb2.ArtifactMetadata(
           name=artifact_name,
-          sha256=_get_file_hash(local_path_to_artifact))
+          sha256=_get_file_hash(local_path_to_artifact),
+          permissions=444)
       metadata = beam_artifact_api_pb2.PutArtifactMetadata(
           staging_session_token=self._staging_session_token,
           metadata=artifact_metadata)