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


Reply via email to