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

Kyle Weaver commented on BEAM-8274:
-----------------------------------

Do we still want to do this? Enabling beam_fn_api by default seems like an 
adequate solution, so I'm not sure we want to do a deeper change unless there's 
a real benefit.

> Remove dependency on experiments=beam_fn_api flag from PortableRunner
> ---------------------------------------------------------------------
>
>                 Key: BEAM-8274
>                 URL: https://issues.apache.org/jira/browse/BEAM-8274
>             Project: Beam
>          Issue Type: Improvement
>          Components: sdk-py-core
>            Reporter: Maximilian Michels
>            Priority: Major
>
> The execution path of Python's {{PortableRunner}} depends on the 
> {{experiments=beam_fn_api}} flag. This flag is set automatically but has been 
> broken in the past, e.g. when the {{FlinkRunner}} was subclassed from the 
> {{PortableRunner}} (BEAM-8037).
> We should get rid of the flag, as the only supported execution path for the 
> PortableRunner is the BeamFnApi. Perhaps we can move the flag to the Dataflow 
> runners which may still depend on it.



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

Reply via email to