Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: filter batch duration opt was not propagated correctly #1224

Merged
merged 1 commit into from
Sep 21, 2024

Conversation

chaitanyaprem
Copy link
Collaborator

@chaitanyaprem chaitanyaprem commented Sep 20, 2024

Description

The fix made in release branch to update filter batch duration ticker to 300ms was not propagated correctly in the code that is moved to go-waku.
this PR addresses the gaps in the fix propagation.

Changes

  • option of batch duration passed to FilterManager is not applied correctly in filter manager causing batch loop to run once every 5 seconds.
  • some movement of options from Subscribe to FilterManager constructor.

waku/v2/api/filter/filter.go Outdated Show resolved Hide resolved
@chaitanyaprem chaitanyaprem merged commit 821481f into master Sep 21, 2024
12 checks passed
@chaitanyaprem chaitanyaprem deleted the fix/filter-batch-duration branch September 21, 2024 01:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants