[GitHub] [beam] boyuanzz commented on pull request #11972: [BEAM-9852] Do not create data channel for empty timer descriptor.

2020-06-10 Thread GitBox
boyuanzz commented on pull request #11972: URL: https://github.com/apache/beam/pull/11972#issuecomment-642246885 Thanks for the explanation! Please run `tox -e py3-yapf` to fix pylint. This is an automated message fr

[GitHub] [beam] boyuanzz commented on pull request #11972: [BEAM-9852] Do not create data channel for empty timer descriptor.

2020-06-10 Thread GitBox
boyuanzz commented on pull request #11972: URL: https://github.com/apache/beam/pull/11972#issuecomment-642199648 Thanks, Kyle! I'm curious with which runner we will run into this issue. Based on the proto, the `url` should be required for `ApiServiceDescriptor`: https://github.com/a