Skip to content

Support Nested Schema Evolution in Parquet for Presto #6675. Copy of …#10158

Closed
sathiscode wants to merge 6 commits intoprestodb:masterfrom
sathiscode:nested-column
Closed

Support Nested Schema Evolution in Parquet for Presto #6675. Copy of …#10158
sathiscode wants to merge 6 commits intoprestodb:masterfrom
sathiscode:nested-column

Conversation

@sathiscode
Copy link

Copy of code from #6675 and migrated to support recent code changes.

afranzi pushed a commit to afranzi/presto that referenced this pull request Apr 24, 2018
@afranzi
Copy link

afranzi commented Jun 19, 2018

Hi @sathiscode & @zhenxiao,

Is there any intention of merging this useful code to master? We are using Presto in Production with this Patch and it works fine too.

It would be great if nested fields and schema evolution where supported by default in Presto.

@sathiscode
Copy link
Author

@afranzi I'm not an admin, so don't have permission to merge the code. Waiting for someone to merge the code.

@sajjoseph
Copy link

sajjoseph commented Nov 3, 2018

With #10737, parquet reader was rewritten and the changes in this PR are no longer working. We will have to incorporate these changes into 0.213 codebase.

The other change to be done is the enhancements to the test cases to validate nested schema evolution.

@Pjack
Copy link

Pjack commented Feb 13, 2019

Amazon Athena utilizes this parquet parser and AWS support refuses to apply the fix on their end until this PR is merged. Do you have plan for this? Thanks!

@Pjack
Copy link

Pjack commented Mar 7, 2019

#6675 is closed, does it mean this issue is resolved too?

@stale
Copy link

stale bot commented Sep 3, 2019

This pull request has been automatically marked as stale because it has not had recent activity. If you'd still like this PR merged, please comment on the task, make sure you've addressed reviewer comments, and rebase on the latest master. Thank you for your contributions!

@stale stale bot added the stale label Sep 3, 2019
@stale stale bot closed this Sep 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants