-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Fix more deprecated Parquet API usage #13554
Fix more deprecated Parquet API usage #13554
Conversation
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 |
lib/trino-parquet/src/test/java/io/trino/parquet/TestParquetTimestampUtils.java
Outdated
Show resolved
Hide resolved
lib/trino-parquet/src/test/java/io/trino/parquet/TestTupleDomainParquetPredicate.java
Outdated
Show resolved
Hide resolved
Please drop the references to GH issues in the commit messages |
63b85df
to
562f5e3
Compare
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 |
562f5e3
to
d3cf981
Compare
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 check |
The cla-bot has been summoned, and re-checked this pull request! |
lib/trino-parquet/src/test/java/io/trino/parquet/TestParquetTimestampUtils.java
Show resolved
Hide resolved
Update usage of the following APIs in trino-parquet - Binary - ColumnChunkMetaData - ColumnDescriptor - Statistics
d3cf981
to
5a5e2d3
Compare
Description
Fix more deprecated Parquet API usage:
Binary
ColumnChunkMetadata
ColumnDescriptor
Statistics
lib/trino-parquet
andplugin/trino-hive
connectorFix deprecated 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: