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

[utils/bloom] Dynamic Bloom Filter #452

Merged
merged 12 commits into from
Jan 10, 2024
Merged

[utils/bloom] Dynamic Bloom Filter #452

merged 12 commits into from
Jan 10, 2024

Conversation

patrick-ogrady
Copy link
Contributor

@patrick-ogrady patrick-ogrady commented Jan 5, 2024

Why this should be merged

Rationale: ava-labs/avalanchego#2591

How this works

Modifies reset logic to grow the bloom filter if the number of target items grows.

How this was tested

CI + Load Test

@ceyonur ceyonur added this to the v0.12.10 milestone Jan 8, 2024
@patrick-ogrady patrick-ogrady merged commit d5a8550 into master Jan 10, 2024
8 checks passed
@patrick-ogrady patrick-ogrady deleted the dynamic-bloom branch January 10, 2024 10:30
darioush pushed a commit that referenced this pull request Apr 25, 2024
* base warp backend

* add signature caching

* add docs

* error handling

* pr fixes

* hash unsigned message for key

* quick pr fixes and merge

* save signature instead of whole msg

* use avaGO cache

* rename warpBackend and docs

* fix nits

* Update plugin/evm/warp_backend.go

* Update plugin/evm/warp_backend.go

* fix pr nits

* pr fixes and testing

* type check for caching

* fix imports

* use memdb and remove extra test

* remove unused

* fix imports

* saving message in db and pr fixes

* update copyright

* update backend variable naming

* add comment about saving db vs cache
oxbee pushed a commit to taurusgroup/coreth that referenced this pull request Nov 6, 2024
* update const name

* lower min target items

* fix lint

* add churn multiplier

* latest ago

* nits

* fix e2e

* update GetFilter

* use correct avalanche version

* update [email protected]

* update scripts version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants