@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 is not a bad thing. 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