[ 
https://issues.apache.org/jira/browse/BEAM-8201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17032758#comment-17032758
 ] 

Robert Bradshaw commented on BEAM-8201:
---------------------------------------

I have updated the container contract with the following suggestions: 

* All the endpoint except the provision endpoint are supplied by the provision 
endpoint. Perhaps we should name this the bootstrap endpoint? 
* Perhaps make SEMI_PERSIST_DIR a reserved constant rather than let the runner 
choose it? 
* Add a first argument which is the phase, with the current instantiation being 
prepare_and_run. 

> clean up the current container API
> ----------------------------------
>
>                 Key: BEAM-8201
>                 URL: https://issues.apache.org/jira/browse/BEAM-8201
>             Project: Beam
>          Issue Type: Improvement
>          Components: build-system
>            Reporter: Hannah Jiang
>            Assignee: Robert Bradshaw
>            Priority: Major
>
> From [~robertwb]
> As part of this project, I propose we look at and clean up the current 
> container API before we "release" it as public and stable. IIRC, we currently 
> provide the worker arguments through a combination of (1) environment 
> variables (2) command line parameters to docker and (3) via the provisioning 
> API. It would be good to have a more principled approach to specifying 
> arguments (either all the same way, or if they vary, good reason for doing so 
> rather than by historical accident).



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to