damccorm commented on issue #35934:
URL: https://github.com/apache/beam/issues/35934#issuecomment-5670653129

   > I had a proposed fix to clear the DoFnInvoker cached window args between 
bundles, however without the overall state cache being enabled this would 
result in re-fetching side inputs from the runner for each bundle. This might 
have a negative impact on streaming performance in particular where bundles are 
generally short-lived.
   
   Isn't this necessary for correctness, though? It seems like what we are 
doing today is basically just caching with bad refresh guarantees.
   
   We could consider reenabling the cache by default, though we'd need to solve 
https://github.com/apache/beam/issues/30360


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