nsivabalan edited a comment on pull request #2449:
URL: https://github.com/apache/hudi/pull/2449#issuecomment-779341027


   @Trevor-zhang : sorry, I didn't suggest to close this out. I am also getting 
conversant w/ hive sync in general. So, was trying to clarify few things. 
   
   if I am not wrong, metastore flow(non JDBC flow)  is already supported. Just 
that config value for "hive.metastore.uris" is taken from Hadoop configuration 
and there is no direct way to pass in as as argument. If you intention is to 
add an argument to make it convenient, then this patch is the right approach. 
If not, lets discuss on what exactly are we trying to achieve here. 


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


Reply via email to