bvolpato commented on PR #28253:
URL: https://github.com/apache/beam/pull/28253#issuecomment-1703913184

   Thanks! No major concerns from me -- just one note is that relying on 
PipelineOptions will allow configuring behavior for the entire job.
   
   Sometimes it's better to use a specific configuration class / instance, so 
you could likely have multiple JOINs in the same job and each handle different 
things.


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to