trinodb / trino

Official repository of Trino, the distributed SQL query engine for big data, formerly known as PrestoSQL (https://trino.io)
https://trino.io
Apache License 2.0
10.48k stars 3.02k forks source link

Hive connector name "hive-hadoop2" is a misnomer #5790

Closed findepi closed 3 years ago

findepi commented 4 years ago

"hive-hadoop2" is a misnomer, as the connector is based on Hadoop 3 libraries and can be used with a Hadoop 3-based environment. The name a vestigial from times when we had multiple flavors of the hive connector (hive-hadoop1, hive-hadoop2, ...).

findepi commented 4 years ago

My proposed solution would be to rename the connector to "hive", retaining "hive-hadoop2" as an alias. (see draft at https://github.com/prestosql/presto/pull/5625, where @electrum pointed out we should update the presto/dir/plugin/hive-hadoop2 folder name (maybe leaving old name as a link?))

@electrum what are your thoughts on this?

findepi commented 4 years ago

@electrum what are your thoughts on this?

findepi commented 3 years ago

Seems like w have agreement - https://github.com/trinodb/trino/pull/5625 & https://github.com/trinodb/trino/pull/8166