Skip to content
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

Support partition schema evolution for ORC and Parquet #2933

Merged

Conversation

@kokosing
Copy link
Member

kokosing commented Feb 25, 2020

No description provided.

Copy link
Member Author

kokosing left a comment

@dain AC

@kokosing kokosing force-pushed the kokosing:origin/master/226_schema_evolution branch 2 times, most recently from 7b752e8 to 5da78aa Mar 19, 2020
@dain
dain approved these changes Mar 24, 2020
Copy link
Member

dain left a comment

Looks good to me, but @phd3 had some good comments.

@kokosing

This comment has been minimized.

Copy link
Member Author

kokosing commented Mar 24, 2020

@phd3 Comments addressed. Thank you for the review!

@kokosing kokosing force-pushed the kokosing:origin/master/226_schema_evolution branch 2 times, most recently from 68a8093 to 7602ff6 Mar 24, 2020
@phd3

This comment has been minimized.

Copy link
Member

phd3 commented Mar 26, 2020

Looks good! Thanks for implementing this change @kokosing

@kokosing kokosing mentioned this pull request Mar 26, 2020
7 of 8 tasks complete
@kokosing kokosing force-pushed the kokosing:origin/master/226_schema_evolution branch from 7602ff6 to a93b3b8 Mar 26, 2020
@kokosing kokosing closed this Mar 26, 2020
@kokosing kokosing deleted the kokosing:origin/master/226_schema_evolution branch Mar 26, 2020
@kokosing kokosing merged commit a93b3b8 into prestosql:master Mar 26, 2020
37 checks passed
37 checks passed
maven-checks (1.8)
Details
hive-tests (config-empty)
Details
x (config-empty, suite-1)
Details
maven-checks (11)
Details
hive-tests (config-hdp3)
Details
x (config-empty, suite-2)
Details
maven-checks (13)
Details
x (config-empty, suite-3)
Details
x (config-empty, suite-5)
Details
x (config-empty, suite-6-non-generic)
Details
x (config-empty, suite-7-non-generic)
Details
x (config-empty, suite-8-non-generic)
Details
x (config-hdp3, suite-1)
Details
x (config-hdp3, suite-2)
Details
x (config-hdp3, suite-3)
Details
x (config-hdp3, suite-5)
Details
x (config-cdh5, suite-1)
Details
x (config-cdh5, suite-2)
Details
x (config-cdh5, suite-3)
Details
x (config-cdh5, suite-5)
Details
error-prone-checks
Details
test-other-modules
Details
web-ui-checks
Details
x (presto-main)
Details
x (presto-tests) x (presto-tests)
Details
x (presto-tests -P ci-only)
Details
x (presto-raptor-legacy)
Details
x (presto-accumulo)
Details
x (presto-cassandra)
Details
x (presto-hive,presto-orc)
Details
x (presto-hive,presto-parquet -P test-parquet)
Details
x (presto-mongodb,presto-kafka,presto-elasticsearch)
Details
x (presto-redis)
Details
x (presto-sqlserver,presto-postgresql,presto-mysql)
Details
x (presto-phoenix,presto-iceberg)
Details
x (presto-kudu)
Details
verification/cla-signed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.