etseidl commented on PR #3470: URL: https://github.com/apache/parquet-java/pull/3470#issuecomment-5181260217
> @etseidl What are the plans for this PR? Can we merge it as-is? If not, can we split out the reader change into a separate PR as it's an orthogonal improvement to stop using `path_in_schema` on the read side? @divjotarora this PR is in limbo until the parquet versioning debate is settled. The issue with making only the read side change is that the thrift generated parser will error out until the IDL can be changed to make `path_in_schema` optional. I don't think there's an appetite here to make local changes to parquet.thrift. If I'm wrong, I'm happy to split out the read and write side changes. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
