damccorm commented on PR #39428:
URL: https://github.com/apache/beam/pull/39428#issuecomment-6060967260

   It looks like this has bounced around for a while without review. One reason 
for this is probably that it is so large - 3,600 lines is quite a lot for a 
reviewer to manage. @rahul-madaan could you please split this up into several 
smaller PRs?
   
   I'd probably also recommend writing up a short design document and sharing 
this with Beam's dev list (https://beam.apache.org/community/contact-us/). 
Jumping straight into this much code will be challenging otherwise


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