nsivabalan commented on code in PR #18084:
URL: https://github.com/apache/hudi/pull/18084#discussion_r2849352061


##########
hudi-common/src/main/java/org/apache/hudi/common/util/CommitUtils.java:
##########
@@ -100,6 +100,26 @@ public static HoodieCommitMetadata 
buildMetadata(List<HoodieWriteStat> writeStat
     return commitMetadata;
   }
 
+  /**
+   * Merges engine-specific metadata (e.g., spark_application_id) with the 
provided extra metadata.
+   *
+   * @param extraMetadata the original extra metadata option
+   * @param engineMetadata the engine-specific metadata to merge
+   * @return merged metadata option, or original if engine metadata is empty
+   */
+  public static Option<Map<String, String>> mergeEngineMetadata(

Review Comment:
   can we write some UTs for this



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to