damccorm commented on PR #17015: URL: https://github.com/apache/beam/pull/17015#issuecomment-1195901551
Hey @fernando-wizeline I agree with the upthread comments that @kennknowles and/or @kileys are probably best equipped to review this, its a little outside of my expertise unfortunately. With that said, I expect you might have a better chance of getting a quicker review if you break this PR up into a few smaller PRs - 1,200 lines is a lot to review in one pass and I imagine that might be part of why this one has been held up (I know I'd feel more comfortable reviewing a 200 liner). Maybe it would make sense to split the files in `examples/java/src/test/java/org/apache/beam/examples/complete/game/` into separate PRs? -- 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]
