Skip to content

Fix an issue where localCache is not copied as part of Settings #7385

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 4 commits into from
Jun 26, 2023

Conversation

wu-hui
Copy link
Contributor

@wu-hui wu-hui commented Jun 22, 2023

Fixes: #7360

Also fixes some public api to use the right classes

@changeset-bot
Copy link

changeset-bot bot commented Jun 22, 2023

🦋 Changeset detected

Latest commit: 2a2b57b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@firebase/firestore Patch
firebase Patch
@firebase/firestore-compat Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@wu-hui wu-hui force-pushed the wuandy/PersistentCacheWithEmulator branch from 8ee1650 to 3ea1c10 Compare June 22, 2023 16:14
@wu-hui wu-hui requested a review from a team as a code owner June 22, 2023 16:24
@google-oss-bot
Copy link
Contributor

google-oss-bot commented Jun 22, 2023

Size Report 1

Affected Products

  • @firebase/firestore

    TypeBase (85ccb2e)Merge (d33b2f7)Diff
    browser370 kB370 kB+30 B (+0.0%)
    esm5356 kB356 kB+30 B (+0.0%)
    main568 kB568 kB+30 B (+0.0%)
    module370 kB370 kB+30 B (+0.0%)
    react-native370 kB370 kB+30 B (+0.0%)
  • @firebase/firestore-lite

    TypeBase (85ccb2e)Merge (d33b2f7)Diff
    browser110 kB110 kB+5 B (+0.0%)
    esm5106 kB106 kB+5 B (+0.0%)
    main151 kB151 kB+5 B (+0.0%)
    module110 kB110 kB+5 B (+0.0%)
    react-native110 kB110 kB+5 B (+0.0%)
  • bundle

    12 size changes

    TypeBase (85ccb2e)Merge (d33b2f7)Diff
    firestore (Persistence)301 kB301 kB+30 B (+0.0%)
    firestore (Query Cursors)238 kB238 kB+30 B (+0.0%)
    firestore (Query)236 kB236 kB+30 B (+0.0%)
    firestore (Read data once)223 kB223 kB+30 B (+0.0%)
    firestore (Realtime updates)226 kB226 kB+30 B (+0.0%)
    firestore (Transaction)202 kB202 kB+30 B (+0.0%)
    firestore (Write data)202 kB202 kB+30 B (+0.0%)
    firestore-lite (Query Cursors)82.9 kB82.9 kB+5 B (+0.0%)
    firestore-lite (Query)79.0 kB79.0 kB+5 B (+0.0%)
    firestore-lite (Read data once)61.1 kB61.1 kB+5 B (+0.0%)
    firestore-lite (Transaction)85.9 kB85.9 kB+5 B (+0.0%)
    firestore-lite (Write data)70.6 kB70.6 kB+5 B (+0.0%)

  • firebase

    TypeBase (85ccb2e)Merge (d33b2f7)Diff
    firebase-compat.js773 kB773 kB+30 B (+0.0%)
    firebase-firestore-compat.js339 kB339 kB+30 B (+0.0%)
    firebase-firestore-lite.js117 kB117 kB+5 B (+0.0%)
    firebase-firestore.js429 kB430 kB+30 B (+0.0%)

Test Logs

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

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Jun 22, 2023

Size Analysis Report 1

Affected Products

  • @firebase/firestore

    • AbstractUserDataWriter

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size25.7 kB25.7 kB+30 B (+0.1%)
      size-with-ext-deps95.4 kB95.4 kB+30 B (+0.0%)
    • AggregateField

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size17.8 kB17.8 kB+30 B (+0.2%)
      size-with-ext-deps87.4 kB87.4 kB+30 B (+0.0%)
    • AggregateQuerySnapshot

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size17.8 kB17.9 kB+30 B (+0.2%)
      size-with-ext-deps87.4 kB87.5 kB+30 B (+0.0%)
    • Bytes

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size19.2 kB19.2 kB+30 B (+0.2%)
      size-with-ext-deps88.8 kB88.8 kB+30 B (+0.0%)
    • CACHE_SIZE_UNLIMITED

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size17.7 kB17.7 kB+30 B (+0.2%)
      size-with-ext-deps87.3 kB87.3 kB+30 B (+0.0%)
    • CollectionReference

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size21.3 kB21.3 kB+30 B (+0.1%)
      size-with-ext-deps90.9 kB90.9 kB+30 B (+0.0%)
    • DocumentReference

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size21.2 kB21.3 kB+30 B (+0.1%)
      size-with-ext-deps90.9 kB90.9 kB+30 B (+0.0%)
    • DocumentSnapshot

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size24.7 kB24.8 kB+30 B (+0.1%)
      size-with-ext-deps94.4 kB94.4 kB+30 B (+0.0%)
    • FieldPath

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size20.3 kB20.4 kB+30 B (+0.1%)
      size-with-ext-deps90.0 kB90.0 kB+30 B (+0.0%)
    • FieldValue

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size17.7 kB17.7 kB+30 B (+0.2%)
      size-with-ext-deps87.3 kB87.3 kB+30 B (+0.0%)
    • Firestore

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size17.7 kB17.7 kB+30 B (+0.2%)
      size-with-ext-deps87.2 kB87.3 kB+30 B (+0.0%)
    • FirestoreError

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size17.7 kB17.7 kB+30 B (+0.2%)
      size-with-ext-deps87.3 kB87.3 kB+30 B (+0.0%)
    • GeoPoint

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size18.2 kB18.3 kB+30 B (+0.2%)
      size-with-ext-deps87.8 kB87.9 kB+30 B (+0.0%)
    • LoadBundleTask

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size18.5 kB18.5 kB+30 B (+0.2%)
      size-with-ext-deps88.1 kB88.1 kB+30 B (+0.0%)
    • Query

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size17.8 kB17.8 kB+30 B (+0.2%)
      size-with-ext-deps87.4 kB87.4 kB+30 B (+0.0%)
    • QueryCompositeFilterConstraint

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size20.9 kB21.0 kB+30 B (+0.1%)
      size-with-ext-deps90.6 kB90.6 kB+30 B (+0.0%)
    • QueryConstraint

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size17.7 kB17.7 kB+30 B (+0.2%)
      size-with-ext-deps87.3 kB87.3 kB+30 B (+0.0%)
    • QueryDocumentSnapshot

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size24.7 kB24.8 kB+30 B (+0.1%)
      size-with-ext-deps94.4 kB94.4 kB+30 B (+0.0%)
    • QueryEndAtConstraint

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size36.5 kB36.5 kB+30 B (+0.1%)
      size-with-ext-deps106 kB106 kB+30 B (+0.0%)
    • QueryFieldFilterConstraint

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size42.2 kB42.2 kB+30 B (+0.1%)
      size-with-ext-deps112 kB112 kB+30 B (+0.0%)
    • QueryLimitConstraint

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size18.4 kB18.5 kB+30 B (+0.2%)
      size-with-ext-deps88.0 kB88.1 kB+30 B (+0.0%)
    • QueryOrderByConstraint

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size19.5 kB19.5 kB+30 B (+0.2%)
      size-with-ext-deps89.1 kB89.1 kB+30 B (+0.0%)
    • QuerySnapshot

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size26.7 kB26.7 kB+30 B (+0.1%)
      size-with-ext-deps96.3 kB96.3 kB+30 B (+0.0%)
    • QueryStartAtConstraint

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size36.5 kB36.5 kB+30 B (+0.1%)
      size-with-ext-deps106 kB106 kB+30 B (+0.0%)
    • SnapshotMetadata

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size17.8 kB17.8 kB+30 B (+0.2%)
      size-with-ext-deps87.4 kB87.4 kB+30 B (+0.0%)
    • Timestamp

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size18.8 kB18.9 kB+30 B (+0.2%)
      size-with-ext-deps88.4 kB88.5 kB+30 B (+0.0%)
    • Transaction

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size51.4 kB51.5 kB+30 B (+0.1%)
      size-with-ext-deps121 kB121 kB+30 B (+0.0%)
    • WriteBatch

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size48.2 kB48.2 kB+30 B (+0.1%)
      size-with-ext-deps118 kB118 kB+30 B (+0.0%)
    • _DatabaseId

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size17.7 kB17.7 kB+30 B (+0.2%)
      size-with-ext-deps87.2 kB87.3 kB+30 B (+0.0%)
    • _DocumentKey

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size26.9 kB27.0 kB+30 B (+0.1%)
      size-with-ext-deps96.6 kB96.6 kB+30 B (+0.0%)
    • _EmptyAppCheckTokenProvider

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size17.8 kB17.8 kB+30 B (+0.2%)
      size-with-ext-deps87.4 kB87.4 kB+30 B (+0.0%)
    • _EmptyAuthCredentialsProvider

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size17.7 kB17.7 kB+30 B (+0.2%)
      size-with-ext-deps87.3 kB87.3 kB+30 B (+0.0%)
    • _FieldPath

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size20.1 kB20.1 kB+30 B (+0.1%)
      size-with-ext-deps89.7 kB89.7 kB+30 B (+0.0%)
    • _TestingHooks

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size18.1 kB18.1 kB+30 B (+0.2%)
      size-with-ext-deps87.7 kB87.7 kB+30 B (+0.0%)
    • _cast

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size18.4 kB18.5 kB+30 B (+0.2%)
      size-with-ext-deps88.0 kB88.1 kB+30 B (+0.0%)
    • _debugAssert

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size17.7 kB17.7 kB+30 B (+0.2%)
      size-with-ext-deps87.3 kB87.3 kB+30 B (+0.0%)
    • _isBase64Available

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size17.7 kB17.7 kB+30 B (+0.2%)
      size-with-ext-deps87.3 kB87.3 kB+30 B (+0.0%)
    • _logWarn

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size17.8 kB17.8 kB+30 B (+0.2%)
      size-with-ext-deps87.3 kB87.4 kB+30 B (+0.0%)
    • _validateIsNotUsedTogether

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size17.7 kB17.7 kB+30 B (+0.2%)
      size-with-ext-deps87.3 kB87.3 kB+30 B (+0.0%)
    • addDoc

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size122 kB122 kB+30 B (+0.0%)
      size-with-ext-deps192 kB192 kB+30 B (+0.0%)
    • aggregateFieldEqual

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size18.0 kB18.1 kB+30 B (+0.2%)
      size-with-ext-deps87.6 kB87.6 kB+30 B (+0.0%)
    • aggregateQuerySnapshotEqual

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size34.0 kB34.0 kB+30 B (+0.1%)
      size-with-ext-deps104 kB104 kB+30 B (+0.0%)
    • and

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size43.6 kB43.7 kB+30 B (+0.1%)
      size-with-ext-deps113 kB113 kB+30 B (+0.0%)
    • arrayRemove

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size30.4 kB30.4 kB+30 B (+0.1%)
      size-with-ext-deps100 kB100 kB+30 B (+0.0%)
    • arrayUnion

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size30.4 kB30.4 kB+30 B (+0.1%)
      size-with-ext-deps100 kB100 kB+30 B (+0.0%)
    • average

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size21.3 kB21.4 kB+30 B (+0.1%)
      size-with-ext-deps91.0 kB91.0 kB+30 B (+0.0%)
    • clearIndexedDbPersistence

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size26.4 kB26.5 kB+30 B (+0.1%)
      size-with-ext-deps96.2 kB96.2 kB+30 B (+0.0%)
    • collection

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size22.0 kB22.0 kB+30 B (+0.1%)
      size-with-ext-deps91.6 kB91.7 kB+30 B (+0.0%)
    • collectionGroup

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size21.0 kB21.0 kB+30 B (+0.1%)
      size-with-ext-deps90.6 kB90.6 kB+30 B (+0.0%)
    • connectFirestoreEmulator

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size19.5 kB19.5 kB+30 B (+0.2%)
      size-with-ext-deps89.7 kB89.7 kB+30 B (+0.0%)
    • count

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size17.8 kB17.8 kB+30 B (+0.2%)
      size-with-ext-deps87.4 kB87.4 kB+30 B (+0.0%)
    • deleteDoc

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size113 kB113 kB+30 B (+0.0%)
      size-with-ext-deps183 kB183 kB+30 B (+0.0%)
    • deleteField

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size18.0 kB18.1 kB+30 B (+0.2%)
      size-with-ext-deps87.6 kB87.7 kB+30 B (+0.0%)
    • disableNetwork

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size101 kB101 kB+30 B (+0.0%)
      size-with-ext-deps171 kB171 kB+30 B (+0.0%)
    • doc

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size22.0 kB22.1 kB+30 B (+0.1%)
      size-with-ext-deps91.7 kB91.7 kB+30 B (+0.0%)
    • documentId

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size20.4 kB20.4 kB+30 B (+0.1%)
      size-with-ext-deps90.0 kB90.0 kB+30 B (+0.0%)
    • enableIndexedDbPersistence

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size183 kB183 kB+30 B (+0.0%)
      size-with-ext-deps255 kB255 kB+30 B (+0.0%)
    • enableMultiTabIndexedDbPersistence

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size220 kB220 kB+30 B (+0.0%)
      size-with-ext-deps292 kB292 kB+30 B (+0.0%)
    • enableNetwork

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size101 kB101 kB+30 B (+0.0%)
      size-with-ext-deps171 kB171 kB+30 B (+0.0%)
    • endAt

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size36.5 kB36.6 kB+30 B (+0.1%)
      size-with-ext-deps106 kB106 kB+30 B (+0.0%)
    • endBefore

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size36.5 kB36.6 kB+30 B (+0.1%)
      size-with-ext-deps106 kB106 kB+30 B (+0.0%)
    • ensureFirestoreConfigured

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size17.8 kB17.8 kB+30 B (+0.2%)
      size-with-ext-deps87.4 kB87.4 kB+30 B (+0.0%)
    • executeWrite

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size112 kB112 kB+30 B (+0.0%)
      size-with-ext-deps182 kB182 kB+30 B (+0.0%)
    • getAggregateFromServer

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size108 kB108 kB+30 B (+0.0%)
      size-with-ext-deps178 kB178 kB+30 B (+0.0%)
    • getCountFromServer

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size108 kB108 kB+30 B (+0.0%)
      size-with-ext-deps179 kB179 kB+30 B (+0.0%)
    • getDoc

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size143 kB143 kB+30 B (+0.0%)
      size-with-ext-deps213 kB213 kB+30 B (+0.0%)
    • getDocFromCache

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size93.6 kB93.6 kB+30 B (+0.0%)
      size-with-ext-deps163 kB163 kB+30 B (+0.0%)
    • getDocFromServer

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size143 kB143 kB+30 B (+0.0%)
      size-with-ext-deps213 kB213 kB+30 B (+0.0%)
    • getDocs

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size144 kB144 kB+30 B (+0.0%)
      size-with-ext-deps215 kB215 kB+30 B (+0.0%)
    • getDocsFromCache

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size101 kB101 kB+30 B (+0.0%)
      size-with-ext-deps171 kB171 kB+30 B (+0.0%)
    • getDocsFromServer

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size144 kB144 kB+30 B (+0.0%)
      size-with-ext-deps215 kB215 kB+30 B (+0.0%)
    • getFirestore

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size19.8 kB19.8 kB+30 B (+0.2%)
      size-with-ext-deps97.5 kB97.6 kB+30 B (+0.0%)
    • increment

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size18.5 kB18.6 kB+30 B (+0.2%)
      size-with-ext-deps88.1 kB88.2 kB+30 B (+0.0%)
    • initializeFirestore

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size18.6 kB18.6 kB+30 B (+0.2%)
      size-with-ext-deps88.7 kB88.7 kB+30 B (+0.0%)
    • limit

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size18.6 kB18.6 kB+30 B (+0.2%)
      size-with-ext-deps88.2 kB88.2 kB+30 B (+0.0%)
    • limitToLast

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size18.6 kB18.7 kB+30 B (+0.2%)
      size-with-ext-deps88.2 kB88.3 kB+30 B (+0.0%)
    • loadBundle

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size111 kB111 kB+30 B (+0.0%)
      size-with-ext-deps181 kB181 kB+30 B (+0.0%)
    • memoryEagerGarbageCollector

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size84.1 kB84.2 kB+30 B (+0.0%)
      size-with-ext-deps154 kB154 kB+30 B (+0.0%)
    • memoryLocalCache

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size98.2 kB98.2 kB+30 B (+0.0%)
      size-with-ext-deps168 kB168 kB+30 B (+0.0%)
    • memoryLruGarbageCollector

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size90.6 kB90.7 kB+30 B (+0.0%)
      size-with-ext-deps160 kB161 kB+30 B (+0.0%)
    • namedQuery

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size87.0 kB87.1 kB+30 B (+0.0%)
      size-with-ext-deps157 kB157 kB+30 B (+0.0%)
    • onSnapshot

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size145 kB145 kB+30 B (+0.0%)
      size-with-ext-deps215 kB215 kB+30 B (+0.0%)
    • onSnapshotsInSync

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size134 kB135 kB+30 B (+0.0%)
      size-with-ext-deps205 kB205 kB+30 B (+0.0%)
    • or

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size43.6 kB43.7 kB+30 B (+0.1%)
      size-with-ext-deps113 kB113 kB+30 B (+0.0%)
    • orderBy

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size23.0 kB23.0 kB+30 B (+0.1%)
      size-with-ext-deps92.6 kB92.6 kB+30 B (+0.0%)
    • persistentLocalCache

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size181 kB181 kB+30 B (+0.0%)
      size-with-ext-deps252 kB252 kB+30 B (+0.0%)
    • persistentMultipleTabManager

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size217 kB217 kB+30 B (+0.0%)
      size-with-ext-deps289 kB289 kB+30 B (+0.0%)
    • persistentSingleTabManager

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size180 kB180 kB+30 B (+0.0%)
      size-with-ext-deps252 kB252 kB+30 B (+0.0%)
    • query

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size43.9 kB43.9 kB+30 B (+0.1%)
      size-with-ext-deps114 kB114 kB+30 B (+0.0%)
    • queryEqual

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size33.9 kB33.9 kB+30 B (+0.1%)
      size-with-ext-deps104 kB104 kB+30 B (+0.0%)
    • refEqual

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size21.4 kB21.5 kB+30 B (+0.1%)
      size-with-ext-deps91.1 kB91.1 kB+30 B (+0.0%)
    • runTransaction

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size122 kB122 kB+30 B (+0.0%)
      size-with-ext-deps192 kB192 kB+30 B (+0.0%)
    • serverTimestamp

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size18.0 kB18.0 kB+30 B (+0.2%)
      size-with-ext-deps87.6 kB87.6 kB+30 B (+0.0%)
    • setDoc

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size121 kB121 kB+30 B (+0.0%)
      size-with-ext-deps191 kB191 kB+30 B (+0.0%)
    • setIndexConfiguration

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size89.7 kB89.7 kB+30 B (+0.0%)
      size-with-ext-deps160 kB160 kB+30 B (+0.0%)
    • setLogLevel

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size17.7 kB17.7 kB+30 B (+0.2%)
      size-with-ext-deps87.3 kB87.3 kB+30 B (+0.0%)
    • snapshotEqual

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size39.6 kB39.6 kB+30 B (+0.1%)
      size-with-ext-deps109 kB109 kB+30 B (+0.0%)
    • startAfter

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size36.5 kB36.6 kB+30 B (+0.1%)
      size-with-ext-deps106 kB106 kB+30 B (+0.0%)
    • startAt

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size36.5 kB36.6 kB+30 B (+0.1%)
      size-with-ext-deps106 kB106 kB+30 B (+0.0%)
    • sum

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size21.3 kB21.4 kB+30 B (+0.1%)
      size-with-ext-deps91.0 kB91.0 kB+30 B (+0.0%)
    • terminate

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size17.8 kB17.8 kB+30 B (+0.2%)
      size-with-ext-deps87.6 kB87.6 kB+30 B (+0.0%)
    • updateDoc

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size122 kB122 kB+30 B (+0.0%)
      size-with-ext-deps192 kB192 kB+30 B (+0.0%)
    • waitForPendingWrites

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size102 kB102 kB+30 B (+0.0%)
      size-with-ext-deps172 kB172 kB+30 B (+0.0%)
    • where

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size43.0 kB43.0 kB+30 B (+0.1%)
      size-with-ext-deps113 kB113 kB+30 B (+0.0%)
    • writeBatch

      Size

      TypeBase (85ccb2e)Merge (d33b2f7)Diff
      size123 kB124 kB+30 B (+0.0%)
      size-with-ext-deps194 kB194 kB+30 B (+0.0%)

Test Logs

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

Copy link
Contributor

@MarkDuckworth MarkDuckworth left a comment

Choose a reason for hiding this comment

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

LGTM. As far as I can tell, there's no public API change. Is that right?

@wu-hui
Copy link
Contributor Author

wu-hui commented Jun 26, 2023

LGTM. As far as I can tell, there's no public API change. Is that right?

Yep, that is right.

@wu-hui wu-hui assigned markarndt and unassigned MarkDuckworth Jun 26, 2023
@wu-hui
Copy link
Contributor Author

wu-hui commented Jun 26, 2023

@markarndt Please review the release note, there is also some minor fixes on public docs, some non-existent classes were incorrectly mentioned in the public ref doc.

@wu-hui wu-hui merged commit aaf3fa3 into master Jun 26, 2023
@wu-hui wu-hui deleted the wuandy/PersistentCacheWithEmulator branch June 26, 2023 18:01
@google-oss-bot google-oss-bot mentioned this pull request Jul 6, 2023
@ocavue
Copy link
Contributor

ocavue commented Jul 7, 2023

@wu-hui I apologize for mentioning you in a closed pull request. I noticed this change in the recent v10.0.0 changelog, and I'm having a bit of difficulty understanding the full impact of this bug. Could you kindly clarify whether this means the localCache configuration in an application using the previous major version Firebase (v9.23.0) wasn't effective? Thank you for your assistance!

@wu-hui
Copy link
Contributor Author

wu-hui commented Jul 7, 2023

@wu-hui I apologize for mentioning you in a closed pull request. I noticed this change in the recent v10.0.0 changelog, and I'm having a bit of difficulty understanding the full impact of this bug. Could you kindly clarify whether this means the localCache configuration in an application using the previous major version Firebase (v9.23.0) wasn't effective? Thank you for your assistance!

The issue is if you call connectEmulator after you set localCache, the localCache value is ignored by the SDK. This should only happen in your testing code.

@firebase firebase locked and limited conversation to collaborators Jul 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

IndexeddbLocalCacheSettings is ignored when SDK is also configured to connect to emulator
5 participants