y1chi commented on a change in pull request #12727:
URL: https://github.com/apache/beam/pull/12727#discussion_r492290645



##########
File path: sdks/python/apache_beam/runners/portability/stager.py
##########
@@ -119,6 +119,7 @@ def create_job_resources(options,  # type: PipelineOptions
                            temp_dir,  # type: str
                            build_setup_args=None,  # type: Optional[List[str]]
                            populate_requirements_cache=None,  # type: 
Optional[str]
+                           skip_boot_dependencies=False, # type: Optional[bool]

Review comment:
       Sounds good to me. The pickled main session is expected to be stored in 
semi persistent directory and I believe that is re-mounted at container launch 
time right now. Should we add another flag to boot.go to specify a separate 
path for prestaged picked main session directory?




----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to