-
Notifications
You must be signed in to change notification settings - Fork 928
Cleanup and improvements to the "complex" bloom filter integration test #7112
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
Cleanup and improvements to the "complex" bloom filter integration test #7112
Conversation
|
…docs)) to fix the lint error: https://eslint.org/docs/latest/rules/guard-for-in
Size Report 1Affected ProductsNo changes between base commit (7c20d7d) and merge commit (75ed530).Test Logs |
Size Analysis Report 1Affected ProductsNo changes between base commit (7c20d7d) and merge commit (75ed530).Test Logs |
…lure (patched in from #7088)
FYI I can't figure out the failures of "Test Firestore / Test Firestore on Firefox If Changed (pull_request)". It appears to be completely unrelated to this PR and passes easily if I tweak the scripts to run the tests sequentially rather than in parallel (see the last two commits in this PR). I think you can just ignore that failing check. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This is a follow-up to #7095 to clean up and improve some of the logic added to the 'resuming a query should use bloom filter to avoid full requery' test.
Ported to Android in Ported to Android in firebase/firebase-android-sdk#4768.