nikitagrover19 commented on PR #37071: URL: https://github.com/apache/beam/pull/37071#issuecomment-3748458848
Thanks for the feedback, that makes sense. Given the concerns about value vs risk, I agree this refactor may not be worth pursuing as is. I’ll plan to close this PR for now and, if useful follow up separately with a much smaller change after checking if that would be welcome. Thanks again for the review and for taking the time to explain the reasoning , I really appreciate it as a first-time contributor. -- 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]
