improve: event filtering algorithm for multiple parallel updates#3155
improve: event filtering algorithm for multiple parallel updates#3155csviri merged 6 commits intooperator-framework:nextfrom
Conversation
csviri
changed the title
shawkins
left a comment
shawkins
left a comment
There was a problem hiding this comment.
This makes sense. To drive home the usage pattern you could consider collapsing decreaseActiveUpdates, handleLastOwnUpdatedResourceVersion, and getActiveUpdates to a single call -
int decreaseActiveUpdates(String resourceVersion)
did some simplification; feel free to do additional (in this or separate pr), and mainly thank you for feedback! |
metacosm
left a comment
metacosm
left a comment
There was a problem hiding this comment.
I didn't follow the latest changes in this area so I'm not qualified to assert the validity of this PR.
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
csviri
force-pushed
the
test-double-update
branch
from
f7e28b8 to
818bc0d
Compare
4 participants
Signed-off-by: Attila Mészáros a_meszaros@apple.com