feat: allow creating drop message for accumulator - #358
Conversation
Signed-off-by: Vaibhav Tiwari <vaibhav.tiwari33@gmail.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #358 +/- ##
=======================================
Coverage 92.99% 92.99%
=======================================
Files 68 68
Lines 3656 3656
Branches 236 236
=======================================
Hits 3400 3400
Misses 190 190
Partials 66 66 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Not required anymore: numaproj/numaflow-java#218 (comment) |
vigith
left a comment
There was a problem hiding this comment.
can you create an issue to track for pynumaflow-lite too?
|
May be we should use https://github.com/astral-sh/uv-docker-example/blob/main/standalone.Dockerfile as the reference for all Dockerfiles. It runs final image without uv, no dev dependencies, runs as non-root user, and also sets |
Created a separate issue |
Closes #357
Testing
Tested with and without message drop semantics.
Disk Usage
When using the drop method the size of the folder where WALs are stored remains roughly constant across time:
When not using the drop method and running the loop without awaiting any result, the size steadily increases:
Memory Usage
When using the drop method, the memory usage for the numa container in accumulator vertex remains mostly steady
Without the drop method the memory usage increases quickly