[GitHub] [flink] xuefuz commented on issue #9524: [FLINK-13841][hive] Extend Hive version support to all 1.2 and 2.3 ve…

2019-08-26 Thread GitBox
xuefuz commented on issue #9524: [FLINK-13841][hive] Extend Hive version support to all 1.2 and 2.3 ve… URL: https://github.com/apache/flink/pull/9524#issuecomment-525077757 > LGTM. W.r.t testing, shall we at least add build profiles for these newly added Hive versions in

[GitHub] [flink] xuefuz commented on issue #9524: [FLINK-13841][hive] Extend Hive version support to all 1.2 and 2.3 ve…

2019-08-26 Thread GitBox
xuefuz commented on issue #9524: [FLINK-13841][hive] Extend Hive version support to all 1.2 and 2.3 ve… URL: https://github.com/apache/flink/pull/9524#issuecomment-524974073 PR updated based on review feedback. @bowenli86 could you take another look? Thanks.

[GitHub] [flink] xuefuz commented on issue #9524: [FLINK-13841][hive] Extend Hive version support to all 1.2 and 2.3 ve…

2019-08-23 Thread GitBox
xuefuz commented on issue #9524: [FLINK-13841][hive] Extend Hive version support to all 1.2 and 2.3 ve… URL: https://github.com/apache/flink/pull/9524#issuecomment-524485653 cc @bowenli86 @lirui-apache @zjuwangg This is an