xushiyan commented on PR #9221:
URL: https://github.com/apache/hudi/pull/9221#issuecomment-1704733085

   > The issue above only show up when using new HiveConf(hadoopConf, 
HiveConf.class). When it's reverted it to HiveConf hiveConf = new HiveConf(); 
hiveConf.addResource(hadoopConf); it works fine
   
   @CTTY ok the order of loading resources makes a difference: 1st approach 
first adds `hadoopConf` then overrides whatever from hive-specific confs, 2nd 
approach does the opposite. In our case, we want to respect user-provided 
`hadoopConf` so we should add the resource after `HiveConf` initializes. I'll 
rebase master as the CI seemed not running for your last push.


-- 
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