Skip to content
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

Tomandersen/handshake rewrite #6015

Draft
wants to merge 52 commits into
base: tomandersen/firestoreClientProvider
Choose a base branch
from

Conversation

tom-andersen
Copy link
Contributor

No description provided.

tom-andersen and others added 25 commits May 15, 2024 10:36
Per [b/338407806](https://b.corp.google.com/issues/338407806),

This fixes the issues with the health metrics action failing. It seemed
to stem from a bunch of outdated dependencies in the health metrics sub
projects.
… tomandersen/watchHandshake

# Conflicts:
#	firebase-firestore/src/main/java/com/google/firebase/firestore/remote/RemoteStore.java
…andersen/watchHandshake

# Conflicts:
#	firebase-firestore/src/main/java/com/google/firebase/firestore/remote/RemoteStore.java
Copy link
Contributor

github-actions bot commented Jun 4, 2024

Javadoc Changes:
--- /home/runner/diff/original/firebase-kotlindoc/android/com/google/firebase/firestore/FirebaseFirestore.html	2024-06-19 16:44:01.222773203 +0000
+++ /home/runner/diff/modified/firebase-kotlindoc/android/com/google/firebase/firestore/FirebaseFirestore.html	2024-06-19 16:41:29.828085746 +0000
@@ -167,7 +167,7 @@
             </td>
           </tr>
           <tr>
-            <td><code>synchronized @<a href="https://developer.android.com/reference/kotlin/androidx/annotation/Nullable.html">Nullable</a> <a href="/docs/reference/android/com/google/firebase/firestore/PersistentCacheIndexManager.html">PersistentCacheIndexManager</a></code></td>
+            <td><code>@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/Nullable.html">Nullable</a> <a href="/docs/reference/android/com/google/firebase/firestore/PersistentCacheIndexManager.html">PersistentCacheIndexManager</a></code></td>
             <td>
               <div><code><a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html#getPersistentCacheIndexManager()">getPersistentCacheIndexManager</a>()</code></div>
               <p>Gets the <code>PersistentCacheIndexManager</code> instance used by this <code>FirebaseFirestore</code> object.</p>
@@ -790,7 +790,7 @@
       </div>
       <div class="api-item"><a name="getPersistentCacheIndexManager--"></a><a name="getpersistentcacheindexmanager"></a>
         <h3 class="api-name" id="getPersistentCacheIndexManager()">getPersistentCacheIndexManager</h3>
-        <pre class="api-signature no-pretty-print">synchronized&nbsp;public&nbsp;@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/Nullable.html">Nullable</a> <a href="/docs/reference/android/com/google/firebase/firestore/PersistentCacheIndexManager.html">PersistentCacheIndexManager</a>&nbsp;<a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html#getPersistentCacheIndexManager()">getPersistentCacheIndexManager</a>()</pre>
+        <pre class="api-signature no-pretty-print">public&nbsp;@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/Nullable.html">Nullable</a> <a href="/docs/reference/android/com/google/firebase/firestore/PersistentCacheIndexManager.html">PersistentCacheIndexManager</a>&nbsp;<a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html#getPersistentCacheIndexManager()">getPersistentCacheIndexManager</a>()</pre>
         <p>Gets the <code>PersistentCacheIndexManager</code> instance used by this <code>FirebaseFirestore</code> object. </p>
         <p>This is not the same as Cloud Firestore Indexes. Persistent cache indexes are optional indexes that only exist within the SDK to assist in local query execution.</p>
         <div class="devsite-table-wrapper">

@tom-andersen tom-andersen changed the base branch from tomandersen/dbToken to tomandersen/streamRefactor June 5, 2024 20:04
… tomandersen/handshakeRewrite

# Conflicts:
#	firebase-firestore/src/main/java/com/google/firebase/firestore/remote/WatchStream.java
#	firebase-firestore/src/main/java/com/google/firebase/firestore/remote/WriteStream.java
@google-oss-bot
Copy link
Contributor

Coverage Report 1

Affected Products

No changes between base commit (6d0f288) and merge commit (d5cde9c).

Test Logs

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

@tom-andersen tom-andersen requested a review from wu-hui June 14, 2024 14:04
@google-oss-bot
Copy link
Contributor

google-oss-bot commented Jun 14, 2024

Size Report 1

Affected Products

  • base

    TypeBase (1cbaac1)Merge (46ce541)Diff
    apk (aggressive)?8.80 kB? (?)
    apk (release)?9.77 kB? (?)
  • firebase-firestore

    TypeBase (1cbaac1)Merge (46ce541)Diff
    aar?1.45 MB? (?)
    apk (aggressive)?554 kB? (?)
    apk (release)?11.1 MB? (?)
  • protolite-well-known-types

    TypeBase (1cbaac1)Merge (46ce541)Diff
    aar?1.01 MB? (?)
    apk (aggressive)?134 kB? (?)
    apk (release)?1.68 MB? (?)

Test Logs

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

@tom-andersen tom-andersen changed the base branch from tomandersen/streamRefactor to tomandersen/componentProviderRefactor June 17, 2024 15:58
…actor' into tomandersen/handshakeRewrite

# Conflicts:
#	firebase-firestore/src/androidTest/java/com/google/firebase/firestore/AccessHelper.java
#	firebase-firestore/src/androidTest/java/com/google/firebase/firestore/remote/RemoteStoreTest.java
#	firebase-firestore/src/androidTest/java/com/google/firebase/firestore/testutil/IntegrationTestUtil.java
#	firebase-firestore/src/main/java/com/google/firebase/firestore/FirebaseFirestore.java
#	firebase-firestore/src/main/java/com/google/firebase/firestore/core/FirestoreClient.java
#	firebase-firestore/src/main/java/com/google/firebase/firestore/core/MemoryComponentProvider.java
#	firebase-firestore/src/main/java/com/google/firebase/firestore/remote/FirestoreChannel.java
#	firebase-firestore/src/main/java/com/google/firebase/firestore/remote/RemoteComponenetProvider.java
#	firebase-firestore/src/main/java/com/google/firebase/firestore/remote/RemoteStore.java
#	firebase-firestore/src/main/java/com/google/firebase/firestore/remote/WriteStream.java
#	firebase-firestore/src/test/java/com/google/firebase/firestore/spec/MemorySpecTest.java
#	firebase-firestore/src/test/java/com/google/firebase/firestore/spec/SQLiteSpecTest.java
#	firebase-firestore/src/test/java/com/google/firebase/firestore/spec/SpecTestCase.java
…tomandersen/handshakeRewrite

# Conflicts:
#	firebase-firestore/src/test/java/com/google/firebase/firestore/integration/AsyncTaskAccumulator.java
#	firebase-firestore/src/test/java/com/google/firebase/firestore/integration/FirebaseFirestoreTest.java
@tom-andersen tom-andersen changed the base branch from tomandersen/componentProviderRefactor to tomandersen/handshakeTest June 17, 2024 20:37
…der' into tomandersen/handshakeRewrite

# Conflicts:
#	firebase-firestore/src/androidTest/java/com/google/firebase/firestore/ValidationTest.java
#	firebase-firestore/src/main/java/com/google/firebase/firestore/FirebaseFirestore.java
#	firebase-firestore/src/main/java/com/google/firebase/firestore/FirestoreClientProvider.java
#	firebase-firestore/src/main/java/com/google/firebase/firestore/core/FirestoreClient.java
#	firebase-firestore/src/test/java/com/google/firebase/firestore/FirebaseFirestoreIntegrationTestFactory.java
@tom-andersen tom-andersen changed the base branch from tomandersen/handshakeTest to tomandersen/firestoreClientProvider June 19, 2024 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants