xinyuiscool commented on pull request #14155:
URL: https://github.com/apache/beam/pull/14155#issuecomment-808369943


   Make sense. I updated the DoFnInvokersTest to verify the setup() will be 
invoked with pipelineOptions by DoFnInvoker, but it doesn't verify from the 
runner side. Let me add a ValidatesRunner test there. Thanks for the suggestion.


-- 
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]


Reply via email to