mxm commented on pull request #11777:
URL: https://github.com/apache/beam/pull/11777#issuecomment-638750806
Since the test has been added and this was the missing piece here, I'm going
to merge.
This is an automated message
mxm commented on pull request #11777:
URL: https://github.com/apache/beam/pull/11777#issuecomment-638146606
Please note that the fix commit is the parent of the commit which introduced
the test. So you'll see the test commit fail and the fix commit pass. GitHub
sorts by timestamps which
mxm commented on pull request #11777:
URL: https://github.com/apache/beam/pull/11777#issuecomment-638145542
Probably not. I finally managed to produce a very trimmed down version of
the pipeline for which the execution still stalls.
Please have a look.
mxm commented on pull request #11777:
URL: https://github.com/apache/beam/pull/11777#issuecomment-634610378
This should be covered by the tests added in the PR which introduced the
changes: https://github.com/apache/beam/pull/10304.
mxm commented on pull request #11777:
URL: https://github.com/apache/beam/pull/11777#issuecomment-632572299
Run Python PreCommit
This is an automated message from the Apache Git Service.
To respond to the message, please log
mxm commented on pull request #11777:
URL: https://github.com/apache/beam/pull/11777#issuecomment-632181899
Please have a look @rohdesamuel if you consider the fix valid. I'm not very
familiar with the Python SDK triggering code.
mxm commented on pull request #11777:
URL: https://github.com/apache/beam/pull/11777#issuecomment-632181118
R: @rohdesamuel
This is an automated message from the Apache Git Service.
To respond to the message, please log on