lukecwik commented on pull request #11856:
URL: https://github.com/apache/beam/pull/11856#issuecomment-640693597


   > @tysonjh
   > 
   > I added some tests with 1000 windows. However, there's a problem with 
them: the side input is empty and no elements are accessed. I think it's 
because of wrong windowing strategy: the side input is windowed using 
fixed-time windows, but the main input is not. Is my assumption correct? If so, 
how should we deal with this problem in your opinion?
   
   The main input window is meant to be mapped onto the side input window. This 
is where the runner asks the SDK to map the main window onto the side input 
window: 
https://github.com/apache/beam/blob/fd8d0c61917149844a80f92adf91e5b944b685ec/sdks/python/apache_beam/runners/worker/bundle_processor.py#L1777


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