Skip to content

Firestore: add test bloomFilterShouldCorrectlyEncodeComplexUnicodeCharacters() to QueryTest.java #5135

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 1 commit into from
Jul 6, 2023

Conversation

dconeybe
Copy link
Contributor

@dconeybe dconeybe commented Jul 5, 2023

Add test bloomFilterShouldCorrectlyEncodeComplexUnicodeCharacters() to QueryTest.java. This test verifies that the bloom filter correctly encodes document paths that contain special Unicode characters, like combining characters and characters above 0xFFFF (i.e. characters outside of the BMP, basic multilingual plane).

Googlers see b/289234093 for details.

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

@github-actions
Copy link
Contributor

github-actions bot commented Jul 5, 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.

@google-oss-bot
Copy link
Contributor

Coverage Report 1

Affected Products

  • firebase-firestore

    Overall coverage changed from 44.34% (d68ed5c) to 44.33% (7644bea) by -0.02%.

    FilenameBase (d68ed5c)Merge (7644bea)Diff
    DeleteMutation.java90.48%95.24%+4.76%
    LruGarbageCollector.java97.27%93.64%-3.64%
    SetMutation.java97.22%94.44%-2.78%

Test Logs

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

@github-actions
Copy link
Contributor

github-actions bot commented Jul 5, 2023

Unit Test Results

   162 files   -    656     162 suites   - 656   2m 8s ⏱️ - 35m 2s
1 164 tests  - 3 792  1 148 ✔️  - 3 786  16 💤  -   5  0  - 1 
2 328 runs   - 7 493  2 296 ✔️  - 7 482  32 💤  - 10  0  - 1 

Results for commit 522d154. ± Comparison against base commit d68ed5c.

@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
    Percentiled68ed5c7644beaDiffSignificant (?)
    p10325 ±12 μs336 ±9.6 μs+10.4 μs (+3.2%)NO
    p25340 ±19 μs352 ±10 μs+11.6 μs (+3.4%)NO
    p50359 ±25 μs406 ±33 μs+46.3 μs (+12.9%)NO
    p75420 ±85 μs539 ±57 μs+119 μs (+28.4%)NO
    p90491 ±142 μs650 ±80 μs+159 μs (+32.4%)NO

    20 test runs in comparison
    CommitTest Runs
    d68ed5c
    • 2023-07-05_17:40:30.357762_Dhtn
    • 2023-07-05_17:40:30.360914_hzLo
    • 2023-07-05_17:40:30.360929_JifQ
    • 2023-07-05_17:40:30.360936_iTXC
    • 2023-07-05_17:40:30.360942_EVvn
    • 2023-07-05_17:40:30.360948_owUJ
    • 2023-07-05_17:40:30.360957_iRLA
    • 2023-07-05_17:40:30.360962_dmzq
    • 2023-07-05_17:40:30.360967_GpDv
    • 2023-07-05_17:40:30.360972_QuFC
    7644bea
    • 2023-07-05_20:37:36.279567_omfs
    • 2023-07-05_20:37:36.332666_LZUP
    • 2023-07-05_20:37:36.332971_dzgS
    • 2023-07-05_20:37:36.332978_OjPd
    • 2023-07-05_20:37:36.332985_WQkp
    • 2023-07-05_20:37:36.332991_lrTu
    • 2023-07-05_20:37:36.333000_pusE
    • 2023-07-05_20:37:36.333006_GsAL
    • 2023-07-05_20:37:36.333011_JffJ
    • 2023-07-05_20:37:36.333017_TvyT
    redfin-30
    Percentiled68ed5c7644beaDiffSignificant (?)
    p10606 ±28 μs621 ±34 μs+15.7 μs (+2.6%)NO
    p25621 ±33 μs643 ±41 μs+21.5 μs (+3.5%)NO
    p50650 ±52 μs670 ±49 μs+20.1 μs (+3.1%)NO
    p75681 ±62 μs713 ±59 μs+32.1 μs (+4.7%)NO
    p90725 ±94 μs776 ±82 μs+51.6 μs (+7.1%)NO

    20 test runs in comparison
    CommitTest Runs
    d68ed5c
    • 2023-07-05_17:40:30.357762_Dhtn
    • 2023-07-05_17:40:30.360914_hzLo
    • 2023-07-05_17:40:30.360929_JifQ
    • 2023-07-05_17:40:30.360936_iTXC
    • 2023-07-05_17:40:30.360942_EVvn
    • 2023-07-05_17:40:30.360948_owUJ
    • 2023-07-05_17:40:30.360957_iRLA
    • 2023-07-05_17:40:30.360962_dmzq
    • 2023-07-05_17:40:30.360967_GpDv
    • 2023-07-05_17:40:30.360972_QuFC
    7644bea
    • 2023-07-05_20:37:36.279567_omfs
    • 2023-07-05_20:37:36.332666_LZUP
    • 2023-07-05_20:37:36.332971_dzgS
    • 2023-07-05_20:37:36.332978_OjPd
    • 2023-07-05_20:37:36.332985_WQkp
    • 2023-07-05_20:37:36.332991_lrTu
    • 2023-07-05_20:37:36.333000_pusE
    • 2023-07-05_20:37:36.333006_GsAL
    • 2023-07-05_20:37:36.333011_JffJ
    • 2023-07-05_20:37:36.333017_TvyT
  • timeToInitialDisplay

    DeviceStatisticsDistributions
    oriole-32
    Percentiled68ed5c7644beaDiffSignificant (?)
    p10198 ±3 ms204 ±3 ms+5.16 ms (+2.6%)NO
    p25204 ±4 ms210 ±3 ms+6.36 ms (+3.1%)NO
    p50212 ±4 ms218 ±3 ms+6.09 ms (+2.9%)NO
    p75220 ±4 ms229 ±5 ms+8.96 ms (+4.1%)NO
    p90229 ±3 ms245 ±13 ms+15.8 ms (+6.9%)NO

    20 test runs in comparison
    CommitTest Runs
    d68ed5c
    • 2023-07-05_17:40:30.357762_Dhtn
    • 2023-07-05_17:40:30.360914_hzLo
    • 2023-07-05_17:40:30.360929_JifQ
    • 2023-07-05_17:40:30.360936_iTXC
    • 2023-07-05_17:40:30.360942_EVvn
    • 2023-07-05_17:40:30.360948_owUJ
    • 2023-07-05_17:40:30.360957_iRLA
    • 2023-07-05_17:40:30.360962_dmzq
    • 2023-07-05_17:40:30.360967_GpDv
    • 2023-07-05_17:40:30.360972_QuFC
    7644bea
    • 2023-07-05_20:37:36.279567_omfs
    • 2023-07-05_20:37:36.332666_LZUP
    • 2023-07-05_20:37:36.332971_dzgS
    • 2023-07-05_20:37:36.332978_OjPd
    • 2023-07-05_20:37:36.332985_WQkp
    • 2023-07-05_20:37:36.332991_lrTu
    • 2023-07-05_20:37:36.333000_pusE
    • 2023-07-05_20:37:36.333006_GsAL
    • 2023-07-05_20:37:36.333011_JffJ
    • 2023-07-05_20:37:36.333017_TvyT
    redfin-30
    Percentiled68ed5c7644beaDiffSignificant (?)
    p10246 ±3 ms267 ±3 ms+21.3 ms (+8.7%)YES
    p25252 ±2 ms273 ±4 ms+21.5 ms (+8.5%)YES
    p50259 ±3 ms281 ±5 ms+22.2 ms (+8.6%)YES
    p75267 ±3 ms291 ±8 ms+24.3 ms (+9.1%)MAYBE
    p90278 ±4 ms306 ±14 ms+28.4 ms (+10.2%)NO

    20 test runs in comparison
    CommitTest Runs
    d68ed5c
    • 2023-07-05_17:40:30.357762_Dhtn
    • 2023-07-05_17:40:30.360914_hzLo
    • 2023-07-05_17:40:30.360929_JifQ
    • 2023-07-05_17:40:30.360936_iTXC
    • 2023-07-05_17:40:30.360942_EVvn
    • 2023-07-05_17:40:30.360948_owUJ
    • 2023-07-05_17:40:30.360957_iRLA
    • 2023-07-05_17:40:30.360962_dmzq
    • 2023-07-05_17:40:30.360967_GpDv
    • 2023-07-05_17:40:30.360972_QuFC
    7644bea
    • 2023-07-05_20:37:36.279567_omfs
    • 2023-07-05_20:37:36.332666_LZUP
    • 2023-07-05_20:37:36.332971_dzgS
    • 2023-07-05_20:37:36.332978_OjPd
    • 2023-07-05_20:37:36.332985_WQkp
    • 2023-07-05_20:37:36.332991_lrTu
    • 2023-07-05_20:37:36.333000_pusE
    • 2023-07-05_20:37:36.333006_GsAL
    • 2023-07-05_20:37:36.333011_JffJ
    • 2023-07-05_20:37:36.333017_TvyT

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

@dconeybe dconeybe marked this pull request as ready for review July 6, 2023 01:20
@dconeybe dconeybe requested a review from milaGGL July 6, 2023 01:20
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 52ca562 into master Jul 6, 2023
@dconeybe dconeybe deleted the dconeybe/BloomFilterUnicodeTest branch July 6, 2023 15:42
@firebase firebase locked and limited conversation to collaborators Aug 6, 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