hongtao12310 commented on issue #9237: [FLINK-13431][hive] NameNode HA 
configuration was not loaded when running HiveConnector on Yarn
URL: https://github.com/apache/flink/pull/9237#issuecomment-516211740
 
 
   > For test, is it possible to add the following in HiveCatalogITCase.java? 
We can do sth like this:
   > 
   > 1. Create a HiveCatalog.
   > 2. Get the HiveConf object that backs HiveCatalog (Add package 
getHiveConf() in HiveCatalog if necessary.
   > 3. Verify if HDFS configs are in HiveConf.
   > 
   > Otherwise, we can skip for now if it's too complicated.
   > 
   > Thanks.
   
   @xuefuz according to lirui's comment I have put the load HDFS configuration 
logic into HiveConf. so we don't need to create HiveCatalog even if we want to 
add the tests
   
    

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to