nada-attia commented on code in PR #18064:
URL: https://github.com/apache/hudi/pull/18064#discussion_r2761291695
##########
hudi-sync/hudi-hive-sync/src/main/java/org/apache/hudi/hive/HoodieHiveSyncClient.java:
##########
@@ -88,12 +90,12 @@ public HoodieHiveSyncClient(HiveSyncConfig config,
HoodieTableMetaClient metaCli
// disable jdbc and depend on metastore client for all hive registrations
try {
this.client = IMetaStoreClientUtil.getMSC(config.getHiveConf());
+ setMetaConf(config.getHiveConf());
Review Comment:
this ensures HMS registration events have the metafields that are needed to
further identify where these events are coming from i.e. execution engine,
job/pipeline id, etc. otherwise these metafields will show up as NULL when
querying HMS logs.
whenever, we create the client we should setting these metafields for the
client as well.
--
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]
