apache · solrbot · Sep 2, 2026 · Sep 2, 2026 · Sep 3, 2026 · Sep 3, 2026
diff --git a/changelog/unreleased/PR#4847-update-error-prone-to-v2-50-0.yml b/changelog/unreleased/PR#4847-update-error-prone-to-v2-50-0.yml
@@ -0,0 +1,7 @@
+title: Update Error Prone to v2.50.0
+type: dependency_update
+authors:
+- name: solrbot
+links:
+- name: PR#4847
+  url: https://github.com/apache/solr/pull/4847
diff --git a/gradle/libs.versions.toml b/gradle/libs.versions.toml
@@ -99,7 +99,7 @@ google-api-grpc-proto = "2.75.0"
 # @keep for version alignment
 google-autovalue = "1.11.1"
 google-cloud-bom = "0.268.0"
-google-errorprone = "2.41.0"
+google-errorprone = "2.50.0"
 # @keep for version alignment
 google-gson = "2.14.0"
 google-guava = "33.7.1-jre"

diff --git a/gradle/validation/error-prone.gradle b/gradle/validation/error-prone.gradle
@@ -589,11 +589,12 @@ allprojects {prj ->
   }
 }
 
-// Exclude generated OpenAPI sources in solrj from Error Prone checks.
+// Exclude generated OpenAPI sources and the vendored noggit JSON parser (org.noggit, not our
+// code/conventions) in solrj from Error Prone checks.
 configure(project(":solr:solrj")) {
   plugins.withId(libs.plugins.ltgt.errorprone.get().pluginId) {
     tasks.withType(JavaCompile) {
-      options.errorprone.excludedPaths = ".*/build/generated/.*"
+      options.errorprone.excludedPaths = ".*/build/generated/.*|.*/org/noggit/.*"
     }
   }
 }
@@ -606,3 +607,13 @@ configure(project(":solr:core")) {
     }
   }
 }
+
+// Exclude vendored Tika sources (copied in to eliminate the Tika dependency) in
+// solr:modules:extraction from Error Prone checks.
+configure(project(":solr:modules:extraction")) {
+  plugins.withId(libs.plugins.ltgt.errorprone.get().pluginId) {
+    tasks.withType(JavaCompile) {
+      options.errorprone.excludedPaths = ".*/org/apache/solr/handler/extraction/fromtika/.*"
+    }
+  }
+}
diff --git a/solr/api/gradle.lockfile b/solr/api/gradle.lockfile
@@ -21,11 +21,11 @@ com.github.kevinstern:software-and-algorithms:1.0=annotationProcessor,errorprone
 com.google.auto.service:auto-service-annotations:1.0.1=annotationProcessor,errorprone,testAnnotationProcessor
 com.google.auto.value:auto-value-annotations:1.11.1=annotationProcessor,errorprone,testAnnotationProcessor
 com.google.auto:auto-common:1.2.2=annotationProcessor,errorprone,testAnnotationProcessor
-com.google.errorprone:error_prone_annotation:2.41.0=annotationProcessor,errorprone,testAnnotationProcessor
+com.google.errorprone:error_prone_annotation:2.50.0=annotationProcessor,errorprone,testAnnotationProcessor
 com.google.errorprone:error_prone_annotations:2.50.0=annotationProcessor,errorprone,jarValidation,testAnnotationProcessor,testCompileClasspath,testRuntimeClasspath
-com.google.errorprone:error_prone_check_api:2.41.0=annotationProcessor,errorprone,testAnnotationProcessor
-com.google.errorprone:error_prone_core:2.41.0=annotationProcessor,errorprone,testAnnotationProcessor
-com.google.googlejavaformat:google-java-format:1.27.0=annotationProcessor,errorprone,testAnnotationProcessor
+com.google.errorprone:error_prone_check_api:2.50.0=annotationProcessor,errorprone,testAnnotationProcessor
+com.google.errorprone:error_prone_core:2.50.0=annotationProcessor,errorprone,testAnnotationProcessor
+com.google.googlejavaformat:google-java-format:1.35.0=annotationProcessor,errorprone,testAnnotationProcessor
 com.google.guava:failureaccess:1.0.3=annotationProcessor,errorprone,jarValidation,testAnnotationProcessor,testCompileClasspath,testRuntimeClasspath
 com.google.guava:guava:33.7.1-jre=annotationProcessor,errorprone,jarValidation,testAnnotationProcessor,testCompileClasspath,testRuntimeClasspath
 com.google.guava:listenablefuture:9999.0-empty-to-avoid-conflict-with-guava=annotationProcessor,errorprone,jarValidation,testAnnotationProcessor,testCompileClasspath,testRuntimeClasspath

diff --git a/solr/benchmark/gradle.lockfile b/solr/benchmark/gradle.lockfile
@@ -17,11 +17,11 @@ com.github.kevinstern:software-and-algorithms:1.0=annotationProcessor,errorprone
 com.google.auto.service:auto-service-annotations:1.0.1=annotationProcessor,errorprone,testAnnotationProcessor
 com.google.auto.value:auto-value-annotations:1.11.1=annotationProcessor,errorprone,testAnnotationProcessor
 com.google.auto:auto-common:1.2.2=annotationProcessor,errorprone,testAnnotationProcessor
-com.google.errorprone:error_prone_annotation:2.41.0=annotationProcessor,errorprone,testAnnotationProcessor
+com.google.errorprone:error_prone_annotation:2.50.0=annotationProcessor,errorprone,testAnnotationProcessor
 com.google.errorprone:error_prone_annotations:2.50.0=annotationProcessor,compileClasspath,errorprone,jarValidation,runtimeClasspath,testAnnotationProcessor,testCompileClasspath,testRuntimeClasspath
-com.google.errorprone:error_prone_check_api:2.41.0=annotationProcessor,errorprone,testAnnotationProcessor
-com.google.errorprone:error_prone_core:2.41.0=annotationProcessor,errorprone,testAnnotationProcessor
-com.google.googlejavaformat:google-java-format:1.27.0=annotationProcessor,errorprone,testAnnotationProcessor
+com.google.errorprone:error_prone_check_api:2.50.0=annotationProcessor,errorprone,testAnnotationProcessor
+com.google.errorprone:error_prone_core:2.50.0=annotationProcessor,errorprone,testAnnotationProcessor
+com.google.googlejavaformat:google-java-format:1.35.0=annotationProcessor,errorprone,testAnnotationProcessor
 com.google.guava:failureaccess:1.0.3=annotationProcessor,compileClasspath,errorprone,jarValidation,runtimeClasspath,testAnnotationProcessor,testCompileClasspath,testRuntimeClasspath
 com.google.guava:guava:33.7.1-jre=annotationProcessor,compileClasspath,errorprone,jarValidation,runtimeClasspath,testAnnotationProcessor,testCompileClasspath,testRuntimeClasspath
 com.google.guava:listenablefuture:9999.0-empty-to-avoid-conflict-with-guava=annotationProcessor,compileClasspath,errorprone,jarValidation,runtimeClasspath,testAnnotationProcessor,testCompileClasspath,testRuntimeClasspath

diff --git a/solr/core/gradle.lockfile b/solr/core/gradle.lockfile
@@ -20,11 +20,11 @@ com.github.stephenc.jcip:jcip-annotations:1.0-1=compileClasspath,jarValidation
 com.google.auto.service:auto-service-annotations:1.0.1=annotationProcessor,errorprone,testAnnotationProcessor
 com.google.auto.value:auto-value-annotations:1.11.1=annotationProcessor,errorprone,testAnnotationProcessor
 com.google.auto:auto-common:1.2.2=annotationProcessor,errorprone,testAnnotationProcessor
-com.google.errorprone:error_prone_annotation:2.41.0=annotationProcessor,errorprone,testAnnotationProcessor
+com.google.errorprone:error_prone_annotation:2.50.0=annotationProcessor,errorprone,testAnnotationProcessor
 com.google.errorprone:error_prone_annotations:2.50.0=annotationProcessor,compileClasspath,errorprone,jarValidation,runtimeClasspath,runtimeLibs,testAnnotationProcessor,testCompileClasspath,testRuntimeClasspath
-com.google.errorprone:error_prone_check_api:2.41.0=annotationProcessor,errorprone,testAnnotationProcessor
-com.google.errorprone:error_prone_core:2.41.0=annotationProcessor,errorprone,testAnnotationProcessor
-com.google.googlejavaformat:google-java-format:1.27.0=annotationProcessor,errorprone,testAnnotationProcessor
+com.google.errorprone:error_prone_check_api:2.50.0=annotationProcessor,errorprone,testAnnotationProcessor
+com.google.errorprone:error_prone_core:2.50.0=annotationProcessor,errorprone,testAnnotationProcessor
+com.google.googlejavaformat:google-java-format:1.35.0=annotationProcessor,errorprone,testAnnotationProcessor
 com.google.guava:failureaccess:1.0.3=annotationProcessor,compileClasspath,errorprone,jarValidation,runtimeClasspath,runtimeLibs,testAnnotationProcessor,testCompileClasspath,testRuntimeClasspath
 com.google.guava:guava:33.7.1-jre=annotationProcessor,compileClasspath,errorprone,jarValidation,runtimeClasspath,runtimeLibs,testAnnotationProcessor,testCompileClasspath,testRuntimeClasspath
 com.google.guava:listenablefuture:9999.0-empty-to-avoid-conflict-with-guava=annotationProcessor,compileClasspath,errorprone,jarValidation,runtimeClasspath,runtimeLibs,testAnnotationProcessor,testCompileClasspath,testRuntimeClasspath

diff --git a/solr/core/src/java/org/apache/solr/cli/ExportTool.java b/solr/core/src/java/org/apache/solr/cli/ExportTool.java
@@ -594,6 +594,8 @@ private void addProducers(Map<String, Slice> m) {
       }
     }
 
+    @SuppressWarnings(
+        "ReferenceEquality") // EOFDOC is a unique sentinel; identity check is intentional
     private void addConsumer(CountDownLatch consumerlatch) {
       consumerThreadpool.execute(
           () -> {

diff --git a/solr/core/src/java/org/apache/solr/cloud/DistributedClusterStateUpdater.java b/solr/core/src/java/org/apache/solr/cloud/DistributedClusterStateUpdater.java
@@ -755,6 +755,8 @@ public boolean isCollectionCreation() {
       }
 
       @Override
+      @SuppressWarnings(
+          "ReferenceEquality") // NO_OP is a unique sentinel; identity check is intentional
       public void computeUpdates(ClusterState clusterState, SolrZkClient client) {
         boolean hasJsonUpdates = false;
         List<PerReplicaStatesOps> perReplicaStateOps = new ArrayList<>();
@@ -912,6 +914,8 @@ public boolean isCollectionCreation() {
     }
 
     @Override
+    @SuppressWarnings(
+        "ReferenceEquality") // NO_OP is a unique sentinel; identity check is intentional
     public void computeUpdates(ClusterState clusterState, SolrZkClient client) {
       final DocCollection docCollection = clusterState.getCollectionOrNull(collectionName);
       Optional<ZkWriteCommand> result =

diff --git a/solr/core/src/java/org/apache/solr/cloud/LockTree.java b/solr/core/src/java/org/apache/solr/cloud/LockTree.java
@@ -214,6 +214,8 @@ boolean isLocked() {
       return false;
     }
 
+    @SuppressWarnings(
+        "ReferenceEquality") // releasing only the exact lock instance held, by identity
     boolean unlock(LockImpl lockObject) {
       if (--refCount > 0) {
         return false;

diff --git a/solr/core/src/java/org/apache/solr/cloud/Overseer.java b/solr/core/src/java/org/apache/solr/cloud/Overseer.java
@@ -782,9 +782,6 @@ public synchronized void start(String id) {
     assert ObjectReleaseTracker.track(this);
   }
 
-  /** Start {@link ClusterSingleton} plugins when we become the leader. */
-
-  /** Stop {@link ClusterSingleton} plugins when we lose leadership. */
   public Stats getStats() {
     return stats;
   }

diff --git a/solr/core/src/java/org/apache/solr/cloud/ReplicateFromLeader.java b/solr/core/src/java/org/apache/solr/cloud/ReplicateFromLeader.java
@@ -64,6 +64,9 @@ public ReplicateFromLeader(CoreContainer cc, String coreName) {
    * @param switchTransactionLog if true, ReplicationHandler will rotate the transaction log once
    *     the replication is done
    */
+  @SuppressWarnings(
+      "ReferenceEquality") // IndexFetchResult constants are singletons; identity check is
+  // intentional
   public void startReplication(boolean switchTransactionLog) {
     try (SolrCore core = cc.getCore(coreName)) {
       if (core == null) {

diff --git a/solr/core/src/java/org/apache/solr/cloud/api/collections/CategoryRoutedAlias.java b/solr/core/src/java/org/apache/solr/cloud/api/collections/CategoryRoutedAlias.java
@@ -80,6 +80,8 @@ public class CategoryRoutedAlias extends RoutedAlias {
   }
 
   @Override
+  @SuppressWarnings(
+      "ReferenceEquality") // Aliases is replaced wholesale, so identity means "possibly changed"
   public boolean updateParsedCollectionAliases(ZkStateReader zkStateReader, boolean contextualize) {
     final Aliases aliases =
         zkStateReader.getAliases(); // note: might be different from last request

diff --git a/solr/core/src/java/org/apache/solr/cloud/api/collections/DimensionalRoutedAlias.java b/solr/core/src/java/org/apache/solr/cloud/api/collections/DimensionalRoutedAlias.java
@@ -212,6 +212,8 @@ protected String getHeadCollectionIfOrdered(AddUpdateCommand cmd) {
    * @return A list of actions across the DRA.
    */
   @Override
+  @SuppressWarnings(
+      "ReferenceEquality") // RoutedAlias dimension identity, not equality, is what matters here
   protected List<Action> calculateActions(String targetCol) {
     String[] routeValues = SEP_MATCHER.split(targetCol);
     // remove the alias name to avoid all manner of off by one errors...

diff --git a/solr/core/src/java/org/apache/solr/cloud/api/collections/TimeRoutedAlias.java b/solr/core/src/java/org/apache/solr/cloud/api/collections/TimeRoutedAlias.java
@@ -264,6 +264,8 @@ private static void checkMillis(Instant date) {
   }
 
   @Override
+  @SuppressWarnings(
+      "ReferenceEquality") // Aliases is replaced wholesale, so identity means "possibly changed"
   public boolean updateParsedCollectionAliases(ZkStateReader zkStateReader, boolean contextualize) {
     final Aliases aliases = zkStateReader.getAliases();
     if (this.parsedCollectionsAliases != aliases) {

diff --git a/solr/core/src/java/org/apache/solr/cloud/overseer/ZkStateWriter.java b/solr/core/src/java/org/apache/solr/cloud/overseer/ZkStateWriter.java
@@ -129,6 +129,8 @@ public void updateClusterState(Function<ClusterState, ClusterState> fun) {
    *     a {@link org.apache.zookeeper.KeeperException.BadVersionException} this instance becomes
    *     unusable and must be discarded
    */
+  @SuppressWarnings(
+      "ReferenceEquality") // NO_OP is a unique sentinel; identity check is intentional
   public ClusterState enqueueUpdate(
       ClusterState prevState, List<ZkWriteCommand> cmds, ZkWriteCallback callback)
       throws IllegalStateException, Exception {
@@ -189,6 +191,8 @@ public ClusterState enqueueUpdate(
     return clusterState;
   }
 
+  @SuppressWarnings(
+      "ReferenceEquality") // NO_OP is a unique sentinel; identity check is intentional
   private boolean isNoOps(List<ZkWriteCommand> cmds) {
     for (ZkWriteCommand cmd : cmds) {
       if (cmd != NO_OP) return false;
@@ -229,6 +233,8 @@ public ClusterState writePendingUpdates() throws KeeperException, InterruptedExc
    * @throws KeeperException if any ZooKeeper operation results in an error
    * @throws InterruptedException if the current thread is interrupted
    */
+  @SuppressWarnings(
+      "ReferenceEquality") // detecting "same map instance passed back in" means "flush all"
   public ClusterState writePendingUpdates(
       Map<String, ZkWriteCommand> updates, boolean resetPendingUpdateCounters)
       throws IllegalStateException, KeeperException, InterruptedException {

diff --git a/solr/core/src/java/org/apache/solr/cloud/overseer/ZkWriteCommand.java b/solr/core/src/java/org/apache/solr/cloud/overseer/ZkWriteCommand.java
@@ -49,6 +49,8 @@ public ZkWriteCommand(String name, DocCollection collection) {
   }
 
   @Override
+  @SuppressWarnings(
+      "ReferenceEquality") // NO_OP is a unique sentinel; identity check is intentional
   public String toString() {
     return getClass().getSimpleName() + ": " + (this == NO_OP ? "no-op" : name + "=" + collection);
   }

diff --git a/solr/core/src/java/org/apache/solr/core/CachingDirectoryFactory.java b/solr/core/src/java/org/apache/solr/core/CachingDirectoryFactory.java
@@ -223,6 +223,7 @@ private void removeFromCache(CacheValue v) {
 
   // be sure the method is called with the sync lock on this object
   // returns true if we closed the cacheValue, false if it will be closed later
+  @SuppressWarnings("ReferenceEquality") // CacheValue identity, not equality, is what matters here
   private boolean closeCacheValue(CacheValue cacheValue, Set<CacheValue> deferRemove) {
     log.debug("looking to close {} {}", cacheValue.path, cacheValue.closeEntries);
     List<CloseListener> listeners = closeListeners.remove(cacheValue.directory);
@@ -300,6 +301,7 @@ private static Iterable<CacheValue> sorted(Set<CacheValue> vals) {
     return vals.stream().sorted((a, b) -> b.path.compareTo(a.path)).collect(Collectors.toList());
   }
 
+  @SuppressWarnings("ReferenceEquality") // CacheValue identity, not equality, is what matters here
   private boolean maybeDeferClose(CacheValue maybeDefer) {
     assert maybeDefer.deleteOnClose;
     for (CacheValue maybeChildPath : byPathCache.values()) {
@@ -381,6 +383,8 @@ public boolean exists(String path) throws IOException {
    * java.lang.String, boolean)
    */
   @Override
+  @SuppressWarnings(
+      "ReferenceEquality") // same instance back from filterDirectory means "unfiltered"
   public final Directory get(String path, DirContext dirContext, String rawLockType)
       throws IOException {
     String fullPath = normalize(path);

diff --git a/solr/core/src/java/org/apache/solr/core/CoreContainer.java b/solr/core/src/java/org/apache/solr/core/CoreContainer.java
@@ -1445,6 +1445,7 @@ public CoreSorter getCoreSorter() {
     return coreSorter;
   }
 
+  @SuppressWarnings("ReferenceEquality") // SolrCore identity, not equality, is what matters here
   protected SolrCore registerCore(
       CoreDescriptor cd, SolrCore core, boolean registerInZk, boolean skipRecovery) {
     if (core == null) {

diff --git a/solr/core/src/java/org/apache/solr/core/PluginBag.java b/solr/core/src/java/org/apache/solr/core/PluginBag.java
@@ -229,7 +229,10 @@ public T put(String name, T plugin) {
     return old == null ? null : old.get();
   }
 
-  @SuppressWarnings({"unchecked"})
+  @SuppressWarnings({
+    "unchecked",
+    "ReferenceEquality" // detecting the same plugin instance re-registered vs. a real swap
+  })
   public PluginHolder<T> put(String name, PluginHolder<T> plugin) {
     Boolean registerApi = null; // i.e. register for V2
     Boolean disableV1 = null; // i.e. do *not* register for v1

diff --git a/solr/core/src/java/org/apache/solr/core/SolrCore.java b/solr/core/src/java/org/apache/solr/core/SolrCore.java
@@ -2328,6 +2328,8 @@ public RefCounted<SolrIndexSearcher> getSearcher(
    *
    * <p>This method acquires openSearcherLock - do not call with searchLock held!
    */
+  @SuppressWarnings(
+      "ReferenceEquality") // schema is only ever replaced wholesale, so identity is correct here
   public RefCounted<SolrIndexSearcher> openNewSearcher(
       boolean updateHandlerReopens, boolean realtime) {
     if (isClosed()) { // catch some errors quicker
@@ -2544,6 +2546,8 @@ public RefCounted<SolrIndexSearcher> openNewSearcher(
    *     SolrIndexSearcher}.
    */
   // TODO waitSearcher should be an AtomicReference or something that is a more clear API
+  @SuppressWarnings(
+      "ReferenceEquality") // SolrIndexSearcher identity, not equality, is what matters here
   public RefCounted<SolrIndexSearcher> getSearcher(
       boolean forceNew,
       boolean returnSearcher,
@@ -2830,6 +2834,8 @@ public boolean isReloaded() {
   //
   // onDeckSearchers will also be decremented (it should have been incremented
   // as a result of opening a new searcher).
+  @SuppressWarnings(
+      "ReferenceEquality") // detecting re-registration of the exact same searcher holder
   private void registerSearcher(RefCounted<SolrIndexSearcher> newSearcherHolder) {
     synchronized (searcherLock) {
       try {

diff --git a/solr/core/src/java/org/apache/solr/core/SolrPaths.java b/solr/core/src/java/org/apache/solr/core/SolrPaths.java
@@ -123,6 +123,8 @@ public AllowPathBuilder addPath(String path) {
      * Adds an allowed path. Detects "_ALL_" which means all paths are allowed. Does not detect "*"
      * (not supported as a {@link Path} on Windows), see {@link #addPath(String)}.
      */
+    @SuppressWarnings(
+        "ReferenceEquality") // ALL_PATHS is a unique sentinel; identity check is intentional
     public AllowPathBuilder addPath(Path path) {
       if (path == null) {
         return this;

diff --git a/solr/core/src/java/org/apache/solr/core/SolrResourceLoader.java b/solr/core/src/java/org/apache/solr/core/SolrResourceLoader.java
@@ -216,6 +216,8 @@ public SolrResourceLoader(Path instanceDir, ClassLoader parent) {
    *
    * @param urls the URLs of files to add
    */
+  @SuppressWarnings(
+      "ReferenceEquality") // ClassLoader has no value-equality; identity check is intentional
   synchronized void addToClassLoader(List<URL> urls) {
     URLClassLoader newLoader = addURLsToClassLoader(classLoader, urls);
     if (newLoader == classLoader) {
@@ -483,6 +485,8 @@ public <T> Class<? extends T> findClass(String cname, Class<T> expectedType) {
    * @param subpackages the packages to be tried if the cname starts with solr.
    * @return the loaded class. An exception is thrown if it fails
    */
+  @SuppressWarnings(
+      "ReferenceEquality") // detecting the default `packages` array vs. a caller-supplied one
   public <T> Class<? extends T> findClass(
       String cname, Class<T> expectedType, String... subpackages) {
     if (subpackages == null || subpackages.length == 0 || subpackages == packages) {
@@ -694,13 +698,18 @@ public <T> boolean addToCoreAware(T obj) {
     }
   }
 
+  @SuppressWarnings(
+      "ReferenceEquality") // detecting whether the same SolrConfig instance is being re-associated
   protected final void setSolrConfig(SolrConfig config) {
     if (this.config != null && this.config != config) {
       throw new IllegalStateException("SolrConfig instance is already associated with this loader");
     }
     this.config = config;
   }
 
+  @SuppressWarnings(
+      "ReferenceEquality") // detecting whether the same CoreContainer instance is being
+  // re-associated
   protected final void setCoreContainer(CoreContainer coreContainer) {
     if (this.coreContainer != null && this.coreContainer != coreContainer) {
       throw new IllegalStateException(

diff --git a/solr/core/src/java/org/apache/solr/handler/ReplicationHandler.java b/solr/core/src/java/org/apache/solr/handler/ReplicationHandler.java
@@ -480,6 +480,8 @@ static Long getCheckSum(Checksum checksum, Path f) {
 
   private volatile IndexFetcher currentIndexFetcher;
 
+  @SuppressWarnings(
+      "ReferenceEquality") // detecting the shared pollingIndexFetcher vs. a one-off, by identity
   public IndexFetchResult doFetch(SolrParams solrParams, boolean forceReplication) {
     String leaderUrl = solrParams.get(LEADER_URL, null);
     if (!indexFetchLock.tryLock()) return IndexFetchResult.LOCK_OBTAIN_FAILED;
@@ -1446,6 +1448,8 @@ public void preClose(SolrCore core) {
         }
 
         @Override
+        @SuppressWarnings(
+            "ReferenceEquality") // detecting the shared pollingIndexFetcher vs. a one-off
         public void postClose(SolrCore core) {
           if (pollingIndexFetcher != null) {
             pollingIndexFetcher.destroy();
@@ -1493,7 +1497,7 @@ public void shutdown() {
    */
   private SolrEventListener getEventListener(final boolean snapshoot, final boolean getCommit) {
     return new SolrEventListener() {
-      /**
+      /*
        * This refreshes the latest replicateable index commit and optionally can create Snapshots as
        * well
        */

diff --git a/solr/core/src/java/org/apache/solr/handler/admin/CoreAdminHandler.java b/solr/core/src/java/org/apache/solr/handler/admin/CoreAdminHandler.java
@@ -464,6 +464,8 @@ public void submitAsyncTask(TaskObject taskObject) throws SolrException {
       }
     }
 
+    @SuppressWarnings(
+        "ReferenceEquality") // detecting a cache-computed instance vs. our own taskObject
     private void addTask(TaskObject taskObject) {
       // Ensure task ID is not already in use
       TaskObject taskInCache =