tvalentyn commented on pull request #13142: URL: https://github.com/apache/beam/pull/13142#issuecomment-716887038
Thanks for your help, @tszerszen . 1. DATAFLOW_SDK_TARBALL_FILE != WORKFLOW_TARBALL_FILE. DATAFLOW_SDK_TARBALL_FILE is the name used to stage sources of apache beam sdk to staging location. WORKFLOW_TARBALL_FILE is a tarball file representing sources of the pipeline in case the pipeline is wrapped into a python package. 2. It would be easier to review if you separated yapf changes in a separate commit. Are changes in groupby_test.py relevant? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
