tvalentyn commented on issue #28716:
URL: https://github.com/apache/beam/issues/28716#issuecomment-1761770326

   Thanks!
   
   The implementation of GroupIntoBatches includes a processing-time timer: 
https://github.com/apache/beam/blob/3a45ecf4b271997b5ce03e1181676356eaa351e1/sdks/python/apache_beam/transforms/util.py#L1067
 , you can find more information about timers in 
https://beam.apache.org/documentation/programming-guide/#processing-time-timers.
   
   This logic was added in https://github.com/apache/beam/pull/13144
   
   @robertwb do you know if this restriction still applies:  
https://github.com/apache/beam/blob/223dded769df48270df317868dc32144ec2fb353/sdks/python/apache_beam/runners/direct/direct_runner.py#L110-L111
 ? 
   
   We could also try to lift it, run the tests and see what fails.
   


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to