You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In #7794, @langmartin pointed out that the volume watcher's batcher doesn't have a maximum batch size. This code was lifted from our deployment watcher, so we're missing that there as well. In practice, the batch window is only 250ms so the batch sizes never get large enough with deployment watcher for it to become an issue. But it might be with large jobs for volume watcher, and it's a good safety/robustness check.
The text was updated successfully, but these errors were encountered:
I'm going to lock this issue because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.
In #7794, @langmartin pointed out that the volume watcher's batcher doesn't have a maximum batch size. This code was lifted from our deployment watcher, so we're missing that there as well. In practice, the batch window is only 250ms so the batch sizes never get large enough with deployment watcher for it to become an issue. But it might be with large jobs for volume watcher, and it's a good safety/robustness check.
The text was updated successfully, but these errors were encountered: