I also took the liberty of reviewing this change because recently I was looking 
closely at what is going on with IO Performance Tests and searched for a 
sensible solution to the problem (still no luck). 

This change LGTM. I think that even without the current problems with python 
setup this should have been done like this. 

[ Full content available at: https://github.com/apache/beam/pull/6355 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to