Fix deprecated Parquet OriginalType#13549
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 cla@trino.io. 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 |
nevillelyh
changed the title
nevillelyh
force-pushed
the
neville/parquet-original-type
branch
from
599bc28 to
156f2ba
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 cla@trino.io. 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 |
...trino-hive/src/test/java/io/trino/plugin/hive/parquet/write/MapKeyValuesSchemaConverter.java
Outdated
Show resolved
Hide resolved
...est/java/io/trino/plugin/hive/parquet/write/SingleLevelArrayMapKeyValuesSchemaConverter.java
Outdated
Show resolved
Hide resolved
nevillelyh
changed the title
nevillelyh
force-pushed
the
neville/parquet-original-type
branch
from
156f2ba to
32ee7db
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 cla@trino.io. 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 |
- Replace OriginalType with LogicalTypeAnnotation whenever possible - Kept parts of MessageTypeConverter and MetadataReader since they're needed for compatibility reasons
nevillelyh
force-pushed
the
neville/parquet-original-type
branch
from
32ee7db to
c64dca8
Compare
|
@raunaqmorarka CLA approved & everything passed excepted one due to possibly network issue? |
I've triggered a re-run of that job |
4 participants
Description
Replace deprecated Parquet
OriginalTypewithLogicalTypeAnnotationlib/trino-parquetandplugin/trino-hiveconnectorFix 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: