Skip to content

Remove legacy parquet reader#18639

Merged
raunaqmorarka merged 1 commit intotrinodb:masterfrom
raunaqmorarka:pqr-remove
Aug 12, 2023
Merged

Remove legacy parquet reader#18639
raunaqmorarka merged 1 commit intotrinodb:masterfrom
raunaqmorarka:pqr-remove

Conversation

@raunaqmorarka
Copy link
Copy Markdown
Member

@raunaqmorarka raunaqmorarka commented Aug 11, 2023

Description

Remove legacy parquet reader

Additional context and related issues

Release notes

( ) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
(x) Release notes are required, with the following suggested text:

# Hive, Hudi, Iceberg, Delta
* Removed legacy parquet reader. The catalog configuration properties `parquet.optimized-reader.enabled` and `parquet.optimized-nested-reader.enabled` are now defunct and must be removed from server configurations. ({issue}`18639`)

@cla-bot cla-bot bot added the cla-signed label Aug 11, 2023
@github-actions github-actions bot added docs tests:hive hudi Hudi connector iceberg Iceberg connector delta-lake Delta Lake connector hive Hive connector labels Aug 11, 2023
Copy link
Copy Markdown
Member

@findepi findepi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(skimmed)

@wendigo
Copy link
Copy Markdown
Contributor

wendigo commented Aug 11, 2023

Failures seems related

@raunaqmorarka raunaqmorarka force-pushed the pqr-remove branch 2 times, most recently from 7c941c6 to ee0431e Compare August 11, 2023 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed delta-lake Delta Lake connector docs hive Hive connector hudi Hudi connector iceberg Iceberg connector

Development

Successfully merging this pull request may close these issues.

4 participants