rdblue commented on pull request #1558:
URL: https://github.com/apache/iceberg/pull/1558#issuecomment-706613051


   > From hiveConfDir, it should be URI which contains scheme and full path. 
This means it can be loaded by Hadoop Filesystem.
   
   Can you help me understand Flink's behavior a bit more?
   
   * If `hiveConfDir` is a Flink option, then why doesn't Flink automatically 
load the `hive-site.xml` file when it creates a `Configuration`? It doesn't 
seem like this should be delegated to libraries that run inside Flink. I would 
expect Flink to create the `Configuration` based on application options and 
pass that to libraries.
   * If Flink allows the user to set a separate `hiveConfDir`, why is that not 
just add that directory to the classpath so that `hive-site.xml` can be loaded 
like normal? That's what happens in most Hadoop setup scripts: extra config 
folders are added to the classpath so that the regular method of loading config 
files works.


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

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



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to