trino icon indicating copy to clipboard operation
trino copied to clipboard

Fix deprecated Parquet OriginalType

Open nevillelyh opened this issue 2 years ago • 4 comments

Description

Is this change a fix, improvement, new feature, refactoring, or other?

Replace deprecated Parquet OriginalType with LogicalTypeAnnotation

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)

lib/trino-parquet and plugin/trino-hive connector

How would you describe this change to a non-technical end user or system administrator?

Fix deprecated Parquet API usage

Related issues, pull requests, and links

Part of #1802

Documentation

(x) No documentation is needed. ( ) Sufficient documentation is included in this PR. ( ) Documentation PR is available with #prnumber. ( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

(x) No release notes entries required. ( ) Release notes entries required with the following suggested text:

nevillelyh avatar Aug 08 '22 18:08 nevillelyh

Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to [email protected]. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla

cla-bot[bot] avatar Aug 08 '22 18:08 cla-bot[bot]

Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to [email protected]. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla

cla-bot[bot] avatar Aug 08 '22 19:08 cla-bot[bot]

Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to [email protected]. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla

cla-bot[bot] avatar Aug 09 '22 13:08 cla-bot[bot]

cc @alexjo2144 @electrum

findepi avatar Aug 09 '22 14:08 findepi

@raunaqmorarka CLA approved & everything passed excepted one due to possibly network issue?

nevillelyh avatar Aug 19 '22 04:08 nevillelyh

@raunaqmorarka CLA approved & everything passed excepted one due to possibly network issue?

I've triggered a re-run of that job

raunaqmorarka avatar Aug 19 '22 04:08 raunaqmorarka