kennknowles commented on pull request #14718:
URL: https://github.com/apache/beam/pull/14718#issuecomment-832070801


   I think your change is correct. Essentially reverting the prior change, 
which broke things. On the bug it seems we found a bug in how cleanup timers 
were fired (for just Dataflow? for everyone?) and instead of fixing the bug we 
broke the feature with this check. CC @steveniemitz and @reuvenlax.


-- 
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]


Reply via email to