Eliaaazzz commented on issue #40644: URL: https://github.com/apache/beam/issues/40644#issuecomment-6287597883
Thanks, @tvalentyn! That makes sense, and I appreciate the guidance. I’m happy to take this forward, starting from #40603 where useful and keeping the changes focused. If it turns out to need a broader redesign, I’ll flag that here before expanding the scope. @alisha-1000, thanks for offering to help! Would you be interested in taking on the argument validation and its unit tests: checking that the required sizing functions are provided and rejecting incompatible weight-based and length-based options? I can handle the core fixes and pipeline-level regression coverage, and I’d be happy to review your PR and help with any questions along the way. @tvalentyn, does that split sound reasonable to you? -- 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]
