je-ik commented on pull request #12551: URL: https://github.com/apache/beam/pull/12551#issuecomment-673037113
Hi @mxm, can you please help me elaborate on this? This PR fixes issue I notices, but I'm not sure, why there seems to be timers that are not cleared from the original queue. Moreover, I'm not sure, why there can be null value in the new map (see FIXME in the code). With the precondition check some validatesRunner tests fail. ---------------------------------------------------------------- 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]
