Skip to content

Firestore: QueryTest.java: add a test that resumes a query with existence filter #4799

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

Merged
merged 2 commits into from
Mar 20, 2023

Conversation

dconeybe
Copy link
Contributor

@dconeybe dconeybe commented Mar 18, 2023

This is a port of firebase/firebase-js-sdk#7134

The port to the iOS SDK is firebase/firebase-ios-sdk#10974

@github-actions
Copy link
Contributor

github-actions bot commented Mar 18, 2023

📝 PRs merging into main branch

Our main branch should always be in a releasable state. If you are working on a larger change, or if you don't want this change to see the light of the day just yet, consider using a feature branch first, and only merge into the main branch when the code complete and ready to be released.

Add the 'main-merge-ack' label to your PR to confirm merging into the main branch is intended.

1 similar comment
@github-actions
Copy link
Contributor

📝 PRs merging into main branch

Our main branch should always be in a releasable state. If you are working on a larger change, or if you don't want this change to see the light of the day just yet, consider using a feature branch first, and only merge into the main branch when the code complete and ready to be released.

Add the 'main-merge-ack' label to your PR to confirm merging into the main branch is intended.

@google-oss-bot
Copy link
Contributor

Coverage Report 1

Affected Products

  • firebase-firestore

    Overall coverage changed from 44.16% (8929123) to 44.14% (0f5ad7a) by -0.02%.

    FilenameBase (8929123)Merge (0f5ad7a)Diff
    DeleteMutation.java90.48%95.24%+4.76%
    LocalStore.java100.00%99.39%-0.61%
    LruGarbageCollector.java97.20%93.46%-3.74%
    SetMutation.java97.22%94.44%-2.78%

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/fqIlgxDfeJ.html

@github-actions
Copy link
Contributor

Unit Test Results

   158 files   -    542     158 suites   - 542   1m 55s ⏱️ - 26m 35s
1 116 tests  - 3 736  1 100 ✔️  - 3 728  16 💤  -   5  0  - 3 
2 232 runs   - 5 829  2 200 ✔️  - 5 816  32 💤  - 10  0  - 3 

Results for commit 16904c6. ± Comparison against base commit 8929123.

@google-oss-bot
Copy link
Contributor

@google-oss-bot
Copy link
Contributor

Startup Time Report 1

Note: Layout is sometimes suboptimal due to limited formatting support on GitHub. Please check this report on GCS.

Notes

Startup Times

  • fire-fst

    DeviceStatisticsDistributions
    oriole-32
    Percentile89291230f5ad7aDiffSignificant (?)
    p10365 ±35 μs327 ±14 μs-37.9 μs (-10.4%)NO
    p25399 ±50 μs341 ±22 μs-58.6 μs (-14.7%)NO
    p50435 ±56 μs375 ±59 μs-60.5 μs (-13.9%)NO
    p75498 ±68 μs420 ±96 μs-78.8 μs (-15.8%)NO
    p90592 ±154 μs512 ±127 μs-79.3 μs (-13.4%)NO

    20 test runs in comparison
    CommitTest Runs
    8929123
    • 2023-03-17_18:23:50.886966_TRYr
    • 2023-03-17_18:23:50.890397_PWIz
    • 2023-03-17_18:23:50.890585_SsQb
    • 2023-03-17_18:23:50.890593_jRmh
    • 2023-03-17_18:23:50.890600_OJxn
    • 2023-03-17_18:23:50.890610_BemQ
    • 2023-03-17_18:23:50.890617_TQBU
    • 2023-03-17_18:23:50.890623_FMHU
    • 2023-03-17_18:23:50.890629_qixX
    • 2023-03-17_18:23:50.890635_niYw
    0f5ad7a
    • 2023-03-18_19:20:44.069902_LhgF
    • 2023-03-18_19:20:44.070670_wRXq
    • 2023-03-18_19:20:44.070680_Gqbu
    • 2023-03-18_19:20:44.070685_jbQK
    • 2023-03-18_19:20:44.070691_oPPu
    • 2023-03-18_19:20:44.070696_Fbfg
    • 2023-03-18_19:20:44.070702_ohgG
    • 2023-03-18_19:20:44.070706_EQyy
    • 2023-03-18_19:20:44.070711_szyW
    • 2023-03-18_19:20:44.070717_CSBh
    redfin-30
    Percentile89291230f5ad7aDiffSignificant (?)
    p10628 ±30 μs625 ±27 μs-3.26 μs (-0.5%)NO
    p25643 ±35 μs642 ±32 μs-814 ns (-0.1%)NO
    p50666 ±43 μs666 ±47 μs-492 ns (-0.1%)NO
    p75700 ±57 μs699 ±63 μs-620 ns (-0.1%)NO
    p90740 ±75 μs749 ±88 μs+8.75 μs (+1.2%)NO

    20 test runs in comparison
    CommitTest Runs
    8929123
    • 2023-03-17_18:23:50.886966_TRYr
    • 2023-03-17_18:23:50.890397_PWIz
    • 2023-03-17_18:23:50.890585_SsQb
    • 2023-03-17_18:23:50.890593_jRmh
    • 2023-03-17_18:23:50.890600_OJxn
    • 2023-03-17_18:23:50.890610_BemQ
    • 2023-03-17_18:23:50.890617_TQBU
    • 2023-03-17_18:23:50.890623_FMHU
    • 2023-03-17_18:23:50.890629_qixX
    • 2023-03-17_18:23:50.890635_niYw
    0f5ad7a
    • 2023-03-18_19:20:44.069902_LhgF
    • 2023-03-18_19:20:44.070670_wRXq
    • 2023-03-18_19:20:44.070680_Gqbu
    • 2023-03-18_19:20:44.070685_jbQK
    • 2023-03-18_19:20:44.070691_oPPu
    • 2023-03-18_19:20:44.070696_Fbfg
    • 2023-03-18_19:20:44.070702_ohgG
    • 2023-03-18_19:20:44.070706_EQyy
    • 2023-03-18_19:20:44.070711_szyW
    • 2023-03-18_19:20:44.070717_CSBh
  • timeToInitialDisplay

    DeviceStatisticsDistributions
    oriole-32
    Percentile89291230f5ad7aDiffSignificant (?)
    p10204 ±18 ms194 ±3 ms-10.1 ms (-5.0%)NO
    p25212 ±20 ms200 ±4 ms-12.7 ms (-6.0%)NO
    p50223 ±25 ms207 ±4 ms-16.1 ms (-7.2%)NO
    p75238 ±31 ms217 ±5 ms-21.1 ms (-8.9%)NO
    p90252 ±37 ms231 ±9 ms-21.1 ms (-8.4%)NO

    20 test runs in comparison
    CommitTest Runs
    8929123
    • 2023-03-17_18:23:50.886966_TRYr
    • 2023-03-17_18:23:50.890397_PWIz
    • 2023-03-17_18:23:50.890585_SsQb
    • 2023-03-17_18:23:50.890593_jRmh
    • 2023-03-17_18:23:50.890600_OJxn
    • 2023-03-17_18:23:50.890610_BemQ
    • 2023-03-17_18:23:50.890617_TQBU
    • 2023-03-17_18:23:50.890623_FMHU
    • 2023-03-17_18:23:50.890629_qixX
    • 2023-03-17_18:23:50.890635_niYw
    0f5ad7a
    • 2023-03-18_19:20:44.069902_LhgF
    • 2023-03-18_19:20:44.070670_wRXq
    • 2023-03-18_19:20:44.070680_Gqbu
    • 2023-03-18_19:20:44.070685_jbQK
    • 2023-03-18_19:20:44.070691_oPPu
    • 2023-03-18_19:20:44.070696_Fbfg
    • 2023-03-18_19:20:44.070702_ohgG
    • 2023-03-18_19:20:44.070706_EQyy
    • 2023-03-18_19:20:44.070711_szyW
    • 2023-03-18_19:20:44.070717_CSBh
    redfin-30
    Percentile89291230f5ad7aDiffSignificant (?)
    p10230 ±5 ms251 ±5 ms+21.6 ms (+9.4%)MAYBE
    p25236 ±5 ms257 ±5 ms+21.3 ms (+9.0%)MAYBE
    p50243 ±4 ms264 ±5 ms+21.1 ms (+8.7%)YES
    p75251 ±4 ms273 ±4 ms+22.1 ms (+8.8%)YES
    p90259 ±5 ms285 ±4 ms+25.6 ms (+9.9%)YES

    20 test runs in comparison
    CommitTest Runs
    8929123
    • 2023-03-17_18:23:50.886966_TRYr
    • 2023-03-17_18:23:50.890397_PWIz
    • 2023-03-17_18:23:50.890585_SsQb
    • 2023-03-17_18:23:50.890593_jRmh
    • 2023-03-17_18:23:50.890600_OJxn
    • 2023-03-17_18:23:50.890610_BemQ
    • 2023-03-17_18:23:50.890617_TQBU
    • 2023-03-17_18:23:50.890623_FMHU
    • 2023-03-17_18:23:50.890629_qixX
    • 2023-03-17_18:23:50.890635_niYw
    0f5ad7a
    • 2023-03-18_19:20:44.069902_LhgF
    • 2023-03-18_19:20:44.070670_wRXq
    • 2023-03-18_19:20:44.070680_Gqbu
    • 2023-03-18_19:20:44.070685_jbQK
    • 2023-03-18_19:20:44.070691_oPPu
    • 2023-03-18_19:20:44.070696_Fbfg
    • 2023-03-18_19:20:44.070702_ohgG
    • 2023-03-18_19:20:44.070706_EQyy
    • 2023-03-18_19:20:44.070711_szyW
    • 2023-03-18_19:20:44.070717_CSBh

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/hoPnD7DoAt/index.html

Copy link
Contributor

@milaGGL milaGGL left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dconeybe dconeybe merged commit 47e7a88 into master Mar 20, 2023
@dconeybe dconeybe deleted the dconeybe/QueryExistenceFilterTest branch March 20, 2023 22:39
dconeybe added a commit that referenced this pull request Mar 23, 2023
… existence filter to actually validate the existence filter.

This is a port of firebase/firebase-js-sdk#7149, and builds upon the test added in #4799.
dconeybe added a commit that referenced this pull request Mar 23, 2023
… existence filter to actually validate the existence filter.

This is a port of firebase/firebase-js-sdk#7149, and builds upon the test added in #4799.
dconeybe added a commit that referenced this pull request Mar 23, 2023
… existence filter to actually validate the existence filter.

This is a port of firebase/firebase-js-sdk#7149, and builds upon the test added in #4799.
@firebase firebase locked and limited conversation to collaborators Apr 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants