echauchot commented on PR #22446:
URL: https://github.com/apache/beam/pull/22446#issuecomment-1247789934

   > I took a glance on this change and LGTM for me. Taking into account that 
this PR really improves the performance of some transforms while running it on 
Spark (according to Nexmark results), I believe we have to merge it once all 
tests will be green.
   
   I would like to review this before merging but it is very long and I'm stuck 
on other thinks. I'll try my best to take a look ASAP


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