(iceberg) branch main updated: Core, Spark: Rename read.identifier-fields.rely to identifier-fields.rely (#15495)

[russellspitzer]

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

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


The following commit(s) were added to refs/heads/main by this push:
     new 64e0211e42 Core, Spark: Rename read.identifier-fields.rely to 
identifier-fields.rely (#15495)
64e0211e42 is described below

commit 64e0211e424d2c1c4b3e2153c5a54564d80f95b6
Author: Russell Spitzer <russell.spit...@gmail.com>
AuthorDate: Mon Mar 2 15:21:33 2026 -0600

    Core, Spark: Rename read.identifier-fields.rely to identifier-fields.rely 
(#15495)
---
 core/src/main/java/org/apache/iceberg/TableProperties.java        | 4 ++--
 .../src/main/java/org/apache/iceberg/spark/SparkReadConf.java     | 4 ++--
 .../test/java/org/apache/iceberg/spark/source/TestSparkTable.java | 8 ++++----
 3 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/core/src/main/java/org/apache/iceberg/TableProperties.java 
b/core/src/main/java/org/apache/iceberg/TableProperties.java
index 05f284079e..237c2224a3 100644
--- a/core/src/main/java/org/apache/iceberg/TableProperties.java
+++ b/core/src/main/java/org/apache/iceberg/TableProperties.java
@@ -252,9 +252,9 @@ public class TableProperties {
    * query engines for optimization purposes (e.g. eliminating redundant joins 
or distinct). This is
    * not enforced at write time and does not validate existing data.
    */
-  public static final String READ_IDENTIFIER_FIELDS_RELY = 
"read.identifier-fields.rely";
+  public static final String IDENTIFIER_FIELDS_RELY = "identifier-fields.rely";
 
-  public static final boolean READ_IDENTIFIER_FIELDS_RELY_DEFAULT = false;
+  public static final boolean IDENTIFIER_FIELDS_RELY_DEFAULT = false;
 
   public static final String OBJECT_STORE_ENABLED = 
"write.object-storage.enabled";
   public static final boolean OBJECT_STORE_ENABLED_DEFAULT = false;
diff --git 
a/spark/v4.1/spark/src/main/java/org/apache/iceberg/spark/SparkReadConf.java 
b/spark/v4.1/spark/src/main/java/org/apache/iceberg/spark/SparkReadConf.java
index 0b54e1a691..6d2302b160 100644
--- a/spark/v4.1/spark/src/main/java/org/apache/iceberg/spark/SparkReadConf.java
+++ b/spark/v4.1/spark/src/main/java/org/apache/iceberg/spark/SparkReadConf.java
@@ -361,8 +361,8 @@ public class SparkReadConf {
     return confParser
         .booleanConf()
         .sessionConf(SparkSQLProperties.IDENTIFIER_FIELDS_RELY)
-        .tableProperty(TableProperties.READ_IDENTIFIER_FIELDS_RELY)
-        .defaultValue(TableProperties.READ_IDENTIFIER_FIELDS_RELY_DEFAULT)
+        .tableProperty(TableProperties.IDENTIFIER_FIELDS_RELY)
+        .defaultValue(TableProperties.IDENTIFIER_FIELDS_RELY_DEFAULT)
         .parse();
   }
 }
diff --git 
a/spark/v4.1/spark/src/test/java/org/apache/iceberg/spark/source/TestSparkTable.java
 
b/spark/v4.1/spark/src/test/java/org/apache/iceberg/spark/source/TestSparkTable.java
index 92a6426002..106d908b49 100644
--- 
a/spark/v4.1/spark/src/test/java/org/apache/iceberg/spark/source/TestSparkTable.java
+++ 
b/spark/v4.1/spark/src/test/java/org/apache/iceberg/spark/source/TestSparkTable.java
@@ -74,7 +74,7 @@ public class TestSparkTable extends CatalogTestBase {
     // enabling rely without identifier fields still produces no primary key
     sql(
         "ALTER TABLE %s SET TBLPROPERTIES ('%s' = 'true')",
-        tableName, TableProperties.READ_IDENTIFIER_FIELDS_RELY);
+        tableName, TableProperties.IDENTIFIER_FIELDS_RELY);
     sparkTable = loadSparkTable();
     assertThat(primaryKeys(sparkTable)).isEmpty();
   }
@@ -91,7 +91,7 @@ public class TestSparkTable extends CatalogTestBase {
 
     sql(
         "ALTER TABLE %s SET TBLPROPERTIES ('%s' = 'true')",
-        tableName, TableProperties.READ_IDENTIFIER_FIELDS_RELY);
+        tableName, TableProperties.IDENTIFIER_FIELDS_RELY);
 
     sparkTable = loadSparkTable();
     List<PrimaryKey> pks = primaryKeys(sparkTable);
@@ -110,7 +110,7 @@ public class TestSparkTable extends CatalogTestBase {
     // disabling rely removes the primary key
     sql(
         "ALTER TABLE %s SET TBLPROPERTIES ('%s' = 'false')",
-        tableName, TableProperties.READ_IDENTIFIER_FIELDS_RELY);
+        tableName, TableProperties.IDENTIFIER_FIELDS_RELY);
     sparkTable = loadSparkTable();
     assertThat(primaryKeys(sparkTable)).isEmpty();
   }
@@ -137,7 +137,7 @@ public class TestSparkTable extends CatalogTestBase {
     // session conf rely=false overrides table property rely=true
     sql(
         "ALTER TABLE %s SET TBLPROPERTIES ('%s' = 'true')",
-        tableName, TableProperties.READ_IDENTIFIER_FIELDS_RELY);
+        tableName, TableProperties.IDENTIFIER_FIELDS_RELY);
     withSQLConf(
         ImmutableMap.of(SparkSQLProperties.IDENTIFIER_FIELDS_RELY, "false"),
         () -> assertThat(primaryKeys(loadSparkTable())).isEmpty());