kbuci commented on code in PR #18123:
URL: https://github.com/apache/hudi/pull/18123#discussion_r2886909625
##########
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/config/HoodieWriteConfig.java:
##########
@@ -2698,6 +2696,14 @@ public String getLockHiveTableName() {
return getString(HoodieLockConfig.HIVE_TABLE_NAME);
}
+ public String getApplicationId() {
+ return applicationId;
+ }
+
+ public void setApplicationId(String appId) {
Review Comment:
I ended up making this a config key, and mutating it the same way we do for
configs in ClusteringPlanStrategy (see BaseHoodieClient)
Only alternative approach I can think of is making sure every creation of
LockManager and.or TransactionManager passes the app id (in the lock config) .
And there are ~ dozen occurrences of the latter
--
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]
