New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to Hive 3.0.0 #203

Merged
merged 3 commits into from Feb 12, 2019

Conversation

4 participants
@electrum
Copy link
Member

electrum commented Feb 11, 2019

This fixes reading from Parquet when no columns are selected.

Fixes #201

Depends on prestosql/presto-hive-apache#2

@cla-bot cla-bot bot added the cla-signed label Feb 11, 2019

@dain

dain approved these changes Feb 11, 2019

pom.xml Outdated
@@ -416,7 +416,7 @@
<dependency>
<groupId>io.prestosql.hive</groupId>
<artifactId>hive-apache</artifactId>
<version>2.3.4-1</version>
<version>3.0.0-1-SNAPSHOT</version>

This comment has been minimized.

@findepi

findepi Feb 11, 2019

Member

Hive 3.0 and 3.1 are quite different with regards to timestamp.
Are you going to upgrade to Hive 3.1 later?

This comment has been minimized.

@kokosing

kokosing Feb 11, 2019

Member

See: #198

This comment has been minimized.

@electrum

electrum Feb 11, 2019

Author Member

Yes, but this is an easier intermediate step, needed now to fix the linked Parquet issue. I’m still working through the 3.1 update.

@dain

dain approved these changes Feb 11, 2019

@electrum electrum force-pushed the electrum:hive-update30 branch from b1b5136 to e3bde09 Feb 12, 2019

@electrum electrum force-pushed the electrum:hive-update30 branch from e3bde09 to 08df1f2 Feb 12, 2019

@electrum electrum merged commit 095ceaf into prestosql:master Feb 12, 2019

1 check passed

verification/cla-signed
Details

@electrum electrum deleted the electrum:hive-update30 branch Feb 12, 2019

@electrum electrum added this to the 303 milestone Feb 14, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment