[GitHub] [flink] JingsongLi commented on issue #11304: [FLINK-16418][hive] Hide hive version to avoid user confuse

2020-03-05 Thread GitBox
JingsongLi commented on issue #11304: [FLINK-16418][hive] Hide hive version to avoid user confuse URL: https://github.com/apache/flink/pull/11304#issuecomment-595108866 Merging... This is an automated message from the Apache

[GitHub] [flink] JingsongLi commented on issue #11304: [FLINK-16418][hive] Hide hive version to avoid user confuse

2020-03-05 Thread GitBox
JingsongLi commented on issue #11304: [FLINK-16418][hive] Hide hive version to avoid user confuse URL: https://github.com/apache/flink/pull/11304#issuecomment-595108572 > @bowenli86 @JingsongLi For HDP and CDH users, I think they probably should use the HDP and CDH version Hive jars. For e

[GitHub] [flink] JingsongLi commented on issue #11304: [FLINK-16418][hive] Hide hive version to avoid user confuse

2020-03-04 Thread GitBox
JingsongLi commented on issue #11304: [FLINK-16418][hive] Hide hive version to avoid user confuse URL: https://github.com/apache/flink/pull/11304#issuecomment-595019043 > what's the "hive version" of Hive in CDH or HDP? Is there any users use CDH or HDP client? The users that use CDH

[GitHub] [flink] JingsongLi commented on issue #11304: [FLINK-16418][hive] Hide hive version to avoid user confuse

2020-03-04 Thread GitBox
JingsongLi commented on issue #11304: [FLINK-16418][hive] Hide hive version to avoid user confuse URL: https://github.com/apache/flink/pull/11304#issuecomment-595001501 > LGTM, except that we need documentation stating that > > 1. hive versions will be automatically inferred at runti