mxm commented on issue #10386: [BEAM-8972] Add Jenkins job with Combine test 
for portable Java
URL: https://github.com/apache/beam/pull/10386#issuecomment-600519826
 
 
   > > Your root transform executes in an environment. This is not allowed in 
portability. We introduced READ support to implement cross-language support 
with existing sources, but it's only supported for streaming. For batch, you 
need to use the `BoundedReadViaImpulse` wrapper.
   > 
   > I removed `BoundedReadViaImpulse` in #10576. All Java read transforms are 
now expanded and executed as Splittable DoFns as long as the `beam_fn_api` 
experiment is used.
   
   @lukecwik I think we ought to auto-add the `beam_fn_api` flag in the 
Portable Runner similar as done in https://github.com/apache/beam/pull/9834/, 
this will avoid PRs like this one to manually add the flag. Tracking this here: 
https://jira.apache.org/jira/browse/BEAM-9538

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to