Skip to content

Consolidate encryption parquet tests #7280

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.

Sign up for GitHub

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

Jump to bottom
Closed
Tracked by #7278
alamb opened this issue Mar 13, 2025 · 2 comments · Fixed by #7279
Closed
Tracked by #7278

Consolidate encryption parquet tests #7280

alamb opened this issue Mar 13, 2025 · 2 comments · Fixed by #7279
Labels
development-process Related to development process of arrow-rs parquet Changes to the parquet crate

Comments

@alamb
Copy link
Contributor

alamb commented Mar 13, 2025

I think it would be good to move some of the tests to the "integration test" arrow_reader binary rather than as unit tests so the structure visibility is tested (the rules for tests compiled with --lib is different than external) clearer, but we can do that as a follow on PR

Originally posted by @alamb in #6637 (review)
@alamb alamb mentioned this issue Mar 13, 2025
Open
15 tasks
@alamb alamb added the parquet Changes to the parquet crate label Mar 13, 2025
@alamb
Copy link
Contributor Author

alamb commented Mar 13, 2025

Details: #6637 (comment)

@alamb alamb mentioned this issue Mar 13, 2025
Merged
@rok
Copy link
Member

rok commented Mar 13, 2025

Related: #7279

@alamb alamb mentioned this issue Mar 13, 2025
Merged
@alamb alamb added the development-process Related to development process of arrow-rs label Mar 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
development-process Related to development process of arrow-rs parquet Changes to the parquet crate
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Move Parquet encryption tests into the arrow_reader integration tests adamreeve/arrow-rs
2 participants
@rok @alamb