lirui-apache commented on pull request #16532: URL: https://github.com/apache/flink/pull/16532#issuecomment-885479709
@yiduwangkai Thanks for working on this! To implement real e2e test for hive connector, I agree we should use a real hive cluster (probably via docker). If that's too much work, we can do it as a follow-up and focus on refactoring here. And the new test should follow the convention of e2e tests, e.g. use init script/DDL instead of yaml file as @fsk119 has pointed out. -- 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. To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org