Clirr

googleapis · Feb 10, 2024 · e88d48a · e88d48a
1 parent 3bb84b8
commit e88d48a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/google-cloud-firestore/clirr-ignored-differences.xml b/google-cloud-firestore/clirr-ignored-differences.xml
@@ -17,6 +17,12 @@
 
 <!-- see http://www.mojohaus.org/clirr-maven-plugin/examples/ignored-differences.html -->
 <differences>
+  <!-- ReadTimeTransaction - added abstract modifier to Transaction class-->
+  <difference>
+    <differenceType>3005</differenceType>
+    <className>com/google/cloud/firestore/Transaction</className>
+  </difference>
+
   <!-- Shutdown/Shutdown Now -->
   <difference>
     <differenceType>7012</differenceType>

diff --git a/google-cloud-firestore/src/main/java/com/google/cloud/firestore/Transaction.java b/google-cloud-firestore/src/main/java/com/google/cloud/firestore/Transaction.java
@@ -17,6 +17,7 @@
 package com.google.cloud.firestore;
 
 import com.google.api.core.ApiFuture;
+import com.google.api.core.InternalExtensionOnly;
 import java.util.List;
 import java.util.logging.Logger;
 import javax.annotation.Nonnull;
@@ -28,13 +29,14 @@
  *
  * @see Firestore#runTransaction(Function)
  */
+@InternalExtensionOnly
 public abstract class Transaction extends UpdateBuilder<Transaction> {
 
   private static final Logger LOGGER = Logger.getLogger(Transaction.class.getName());
   private static final String READ_BEFORE_WRITE_ERROR_MSG =
       "Firestore transactions require all reads to be executed before all writes";
 
-  Transaction(FirestoreImpl firestore) {
+  protected Transaction(FirestoreImpl firestore) {
     super(firestore);
   }