Skip to content

Add new goog-listen-tags for BloomFilter #10977

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 41 commits into from
Mar 25, 2023
Merged

Add new goog-listen-tags for BloomFilter #10977

merged 41 commits into from
Mar 25, 2023

Conversation

@milaGGL
Copy link
Contributor

@milaGGL milaGGL commented Mar 21, 2023 •
edited
Loading

#no-changelog
ported from web: firebase/firebase-js-sdk#7107
android: firebase/firebase-android-sdk#4777

milaGGL and others added 30 commits February 23, 2023 17:33
@milaGGL
add unchanged names to existence filter and modify spec test runner
969f2ed
@milaGGL
Merge branch 'mila/BloomFilter' into mila/BloomFilter-add--existence-…
b66b02b 
…filter-class
@milaGGL
use BloomFilter instead of proto
1df9410
@milaGGL
update serializer
1e67033
@milaGGL
add bloomfilter equality test
899e3cc
@milaGGL
Merge branch 'mila/BloomFilter' into mila/BloomFilter-add--existence-…
3faa3e8 
…filter-class
@milaGGL
make unchanged_names optional in proto
468f967
@milaGGL
handle invalid unchanged-names input with BloomFilter::Create
23f0fb1
@milaGGL
format the comments
475fffb
@milaGGL
Merge branch 'mila/BloomFilter' into mila/BloomFilter-add--existence-…
5bda231 
…filter-class
@milaGGL
port spec tests
11158e5
@milaGGL
change unchanged_names to Bloom_filter
05caaeb
@milaGGL
add GetDatabaseId to the targetMetadataProvider
b0bb8bf
@milaGGL
remove the parsing unchanged_names to BloomFilter logic
d56f4df
@milaGGL
sort the actual and expected events in spec test before comparing them
4158352
@milaGGL
Merge branch 'mila/BloomFilter-add--existence-filter-class' into mila…
2b8533c 
…/BloomFilter-apply-bloom-filter-on-existence-filter-mismatch
@milaGGL
Revert "Merge branch 'mila/BloomFilter-add--existence-filter-class' i…
ae930ef 
…nto mila/BloomFilter-apply-bloom-filter-on-existence-filter-mismatch"

This reverts commit 2b8533c, reversing
changes made to 4158352.
@milaGGL
resolve comments
91a42e6
@milaGGL
Merge branch 'mila/BloomFilter' into mila/BloomFilter-add--existence-…
e694551 
…filter-class
@milaGGL
Merge branch 'mila/BloomFilter-add--existence-filter-class' into mila…
db883fb 
…/BloomFilter-apply-bloom-filter-on-existence-filter-mismatch
@milaGGL
format
a2b7408
@milaGGL
log error message
f6b062f
@milaGGL
Merge branch 'mila/BloomFilter' into mila/BloomFilter-apply-bloom-fil…
0de7946 
…ter-on-existence-filter-mismatch
@milaGGL
merge queryPurpose spec test check
01c332b
@milaGGL
add back the expectedCount check get deleted
4e1e489
@dconeybe
re-write CompareBits()
523fd6c
@dconeybe
fix typo
6bbc6b2
@milaGGL
resolve comments
2a71c20
@milaGGL
resolve comments
4a97630
@milaGGL
Update bloom_filter.cc
3eec288
@milaGGL milaGGL self-assigned this Mar 21, 2023
@milaGGL milaGGL changed the title initial code Add new goog-listen-tags for BloomFilter Mar 21, 2023
Base automatically changed from mila/BloomFilter-apply-bloom-filter-on-existence-filter-mismatch to mila/BloomFilter March 23, 2023 20:38
milaGGL
milaGGL commented Mar 24, 2023
View reviewed changes
@milaGGL milaGGL marked this pull request as ready for review March 24, 2023 01:42
@milaGGL milaGGL requested a review from dconeybe March 24, 2023 01:45
dconeybe
dconeybe requested changes Mar 24, 2023
View reviewed changes
@milaGGL milaGGL requested a review from dconeybe March 24, 2023 16:58
dconeybe
dconeybe reviewed Mar 24, 2023
View reviewed changes
@milaGGL milaGGL merged commit 08025ae into mila/BloomFilter Mar 25, 2023
@milaGGL milaGGL deleted the mila/BloomFilter-add-new-goog-listen-tag branch March 25, 2023 00:17
@firebase firebase locked and limited conversation to collaborators Apr 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@dconeybe dconeybe dconeybe approved these changes

Assignees

@milaGGL milaGGL

Labels

api: firestore

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@milaGGL @dconeybe @google-oss-bot