bowenli86 commented on a change in pull request #8205: [FLINK-12238] [hive] Support database related operations in GenericHiveMetastoreCatalog and setup flink-connector-hive module URL: https://github.com/apache/flink/pull/8205#discussion_r279454577
########## File path: flink-connectors/flink-connector-hive/src/main/resources/META-INF/NOTICE ########## @@ -0,0 +1,34 @@ +flink-connector-hive Review comment: I've changed hive-exec to be a test dependency now to not block us for so long since our work on hive table source/sink depends on this PR. I thought it is a not-so-hard problem to solve and include hive-exec when setting up flink-connector-hive but seems I was wrong. So let's leave it as test dependency now and change it to compile dependency whenever necessary later. What do you think? ---------------------------------------------------------------- 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
