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

Split Parquet code into multiple files #3332

Closed
bmcdonald3 opened this issue Jun 14, 2024 · 0 comments · Fixed by #3333
Closed

Split Parquet code into multiple files #3332

bmcdonald3 opened this issue Jun 14, 2024 · 0 comments · Fixed by #3333
Assignees

Comments

@bmcdonald3
Copy link
Contributor

The Parquet code currently exists as a single monolithic file, which makes it difficult to manage and is not good practice.

By splitting the Parquet code into multiple files, each limited to a distinct functionality, we can make the code easier to maintain/keep track of/modify. Initially, I am splitting the code into three files (1) read (2) write (3) utility.

There will be follow up PRs to further improve the readability/maintainability of this code.

@bmcdonald3 bmcdonald3 self-assigned this Jun 14, 2024
github-merge-queue bot pushed a commit that referenced this issue Jun 24, 2024
* Split Parquet into two files

* Split into three files

* Rename OtherParquet to UtilParquet

* Fix Util file usage

* Fix Makefile logic

* Fix Arrow dependency check

* Address feedback from Tess
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant