@aaltay, sure.  just to clarify.   Did you mean move this to top of this 
Setup() function? 

@tvalentyn I might be wrong, but I think relying on sys.version_info[0] is more 
indirect way of verifying things, thus less preferred to me.  And in test file 
a bit code redundancy for self-containing makes sense to me. just my 2 cents. 

[ Full content available at: https://github.com/apache/beam/pull/6587 ]
This message was relayed via gitbox.apache.org for devnull@infra.apache.org

Reply via email to