(polaris) branch main updated: Make unmodified fields final (#3797)

[snazy]

This is an automated email from the ASF dual-hosted git repository.

snazy pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/polaris.git


The following commit(s) were added to refs/heads/main by this push:
     new b362a6685 Make unmodified fields final (#3797)
b362a6685 is described below

commit b362a6685b9d69cad0da6a472361490d2b6752ff
Author: Nándor Kollár <nandorkol...@users.noreply.github.com>
AuthorDate: Thu Feb 19 18:10:57 2026 +0100

    Make unmodified fields final (#3797)
    
    * Make unmodified fields final
    
    * Add SuppressWarnings and give a bit more explanantion
---
 .../src/test/java/org/apache/polaris/spark/NoopDeltaCatalog.java      | 3 +++
 .../apache/polaris/service/config/ResolvedFeaturesConfiguration.java  | 4 ++--
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git 
a/plugins/spark/v3.5/spark/src/test/java/org/apache/polaris/spark/NoopDeltaCatalog.java
 
b/plugins/spark/v3.5/spark/src/test/java/org/apache/polaris/spark/NoopDeltaCatalog.java
index f698615e6..8ca17f61b 100644
--- 
a/plugins/spark/v3.5/spark/src/test/java/org/apache/polaris/spark/NoopDeltaCatalog.java
+++ 
b/plugins/spark/v3.5/spark/src/test/java/org/apache/polaris/spark/NoopDeltaCatalog.java
@@ -32,6 +32,9 @@ import org.apache.spark.sql.connector.catalog.TableChange;
 public class NoopDeltaCatalog extends DelegatingCatalogExtension {
   // This is a mock of isUnityCatalog scala val in
   // org.apache.spark.sql.delta.catalog.DeltaCatalog.
+  // Although it appears unused, in fact it is reflectively used in DeltaHelper
+  // Final modifier can't apply here either, because it is also reflectively 
modified.
+  @SuppressWarnings("unused")
   private boolean isUnityCatalog = false;
 
   @Override
diff --git 
a/runtime/service/src/main/java/org/apache/polaris/service/config/ResolvedFeaturesConfiguration.java
 
b/runtime/service/src/main/java/org/apache/polaris/service/config/ResolvedFeaturesConfiguration.java
index 3aa340fea..606836ed6 100644
--- 
a/runtime/service/src/main/java/org/apache/polaris/service/config/ResolvedFeaturesConfiguration.java
+++ 
b/runtime/service/src/main/java/org/apache/polaris/service/config/ResolvedFeaturesConfiguration.java
@@ -33,8 +33,8 @@ import java.util.stream.Collectors;
 @Priority(1)
 public class ResolvedFeaturesConfiguration implements FeaturesConfiguration {
 
-  private Map<String, String> cleanedDefaults;
-  private Map<String, RealmOverrides> realmOverrides;
+  private final Map<String, String> cleanedDefaults;
+  private final Map<String, RealmOverrides> realmOverrides;
 
   public ResolvedFeaturesConfiguration(RawFeaturesConfiguration raw) {
     this.realmOverrides = raw.realmOverrides();