twalthr commented on issue #9149: [FLINK-13303][hive] Add e2e test for hive connector URL: https://github.com/apache/flink/pull/9149#issuecomment-513204542 Hi @lirui-apache, yes implementing such tests requires extra efforts but we need to do perform these efforts before releasing. Unit tests or ITCases do not cover proper communication with external systems including serialization/deserialization and depencency/class loading issues. We can postpone testing different Hive versions but should at least test one Hive version in-depth. The SQL Client has nothing to do with the Hive functionality. We can test the Hive connector without SQL Client and the core Table API only. If this issue does not cover the task of adding an e2e test, then please rename the issue, open a new one, and give someone else the chance to implement a full test.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
