trino icon indicating copy to clipboard operation
trino copied to clipboard

Fix more deprecated Parquet API usage

Open nevillelyh opened this issue 1 year ago • 3 comments

Description

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

Fix more deprecated Parquet API usage:

  • Binary
  • ColumnChunkMetadata
  • ColumnDescriptor
  • Statistics

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 API usage

Related issues, pull requests, and links

Part of #1802

Documentation

( ) 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

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

# Section
* Fix some things. ({issue}`issuenumber`)

nevillelyh avatar Aug 09 '22 03: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 09 '22 03:08 cla-bot[bot]

Please drop the references to GH issues in the commit messages

raunaqmorarka avatar Aug 09 '22 06:08 raunaqmorarka

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]

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 18 '22 20:08 cla-bot[bot]

@cla-bot check

martint avatar Aug 18 '22 20:08 martint

The cla-bot has been summoned, and re-checked this pull request!

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