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