[GitHub] [beam] boyuanzz commented on pull request #11642: Replace call to .checkpoint() in SDF direct runner to .try_claim(0)

2020-05-28 Thread GitBox
boyuanzz commented on pull request #11642: URL: https://github.com/apache/beam/pull/11642#issuecomment-635680691 > This is a single line change and passing all the tests. If the change make sense can we merge it? (question to @boyuanzz ) I don't think the change is correct. I can

[GitHub] [beam] boyuanzz commented on pull request #11642: Replace call to .checkpoint() in SDF direct runner to .try_claim(0)

2020-05-19 Thread GitBox
boyuanzz commented on pull request #11642: URL: https://github.com/apache/beam/pull/11642#issuecomment-631136611 Hi Ashwin, do you want me to start to review now? This is an automated message from the Apache Git Service. To