Skip to content

Conversation

@xxubai
Copy link
Contributor

@xxubai xxubai commented Apr 28, 2025

Extract ParquetSchemaProducer from ParquetVariantUtil to facilitate code reuse and future modifications.

@xxubai xxubai force-pushed the parquet-variant-visitor branch from f32c68c to 554216e Compare April 30, 2025 02:09
@xxubai xxubai changed the title Parquet: Improved parquet type handling for variant Parquet: Refactor parquet schema handling for variant type May 1, 2025
Copy link
Contributor

@aihuaxu aihuaxu left a comment

Choose a reason for hiding this comment

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

LGTM.

@xxubai xxubai requested a review from rdblue May 7, 2025 06:12
@github-actions
Copy link

github-actions bot commented Jun 7, 2025

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the [email protected] list. Thank you for your contributions.

@github-actions github-actions bot added the stale label Jun 7, 2025
@xxubai
Copy link
Contributor Author

xxubai commented Jun 7, 2025

1

@github-actions github-actions bot removed the stale label Jun 8, 2025
@github-actions
Copy link

github-actions bot commented Jul 9, 2025

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the [email protected] list. Thank you for your contributions.

@github-actions github-actions bot added the stale label Jul 9, 2025
@xxubai xxubai closed this Jul 9, 2025
@xxubai xxubai deleted the parquet-variant-visitor branch July 9, 2025 05:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants