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

huaxingao 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 8c7ffec48f Refactor: populate writeOptions in extraSnapshotMetadata 
using common util (#14604)
8c7ffec48f is described below

commit 8c7ffec48ffd8d51e0ec7c8eb12d9aaeee214212
Author: yingjianwu98 <[email protected]>
AuthorDate: Mon Nov 17 21:10:46 2025 -0800

    Refactor: populate writeOptions in extraSnapshotMetadata using common util 
(#14604)
    
    * small refactor for writeOptions when populating extraSnapshotMetadata to 
use common util
    
    * reformat
    
    ---------
    
    Co-authored-by: Yingjian Wu <[email protected]>
---
 .../src/main/java/org/apache/iceberg/spark/SparkWriteConf.java   | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git 
a/spark/v4.0/spark/src/main/java/org/apache/iceberg/spark/SparkWriteConf.java 
b/spark/v4.0/spark/src/main/java/org/apache/iceberg/spark/SparkWriteConf.java
index c4f43d7b77..96131e0e56 100644
--- 
a/spark/v4.0/spark/src/main/java/org/apache/iceberg/spark/SparkWriteConf.java
+++ 
b/spark/v4.0/spark/src/main/java/org/apache/iceberg/spark/SparkWriteConf.java
@@ -261,13 +261,8 @@ public class SparkWriteConf {
             SparkSQLProperties.SNAPSHOT_PROPERTY_PREFIX));
 
     // Add write options, overriding session configuration if necessary
-    writeOptions.forEach(
-        (key, value) -> {
-          if (key.startsWith(SnapshotSummary.EXTRA_METADATA_PREFIX)) {
-            extraSnapshotMetadata.put(
-                key.substring(SnapshotSummary.EXTRA_METADATA_PREFIX.length()), 
value);
-          }
-        });
+    extraSnapshotMetadata.putAll(
+        PropertyUtil.propertiesWithPrefix(writeOptions, 
SnapshotSummary.EXTRA_METADATA_PREFIX));
 
     return extraSnapshotMetadata;
   }

Reply via email to