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.

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

Implement better tests for ParquetExec #134

Closed
alamb opened this issue Apr 26, 2021 · 5 comments
Closed

Implement better tests for ParquetExec #134

alamb opened this issue Apr 26, 2021 · 5 comments
Labels
datafusion Changes in the datafusion crate good first issue Good for newcomers help wanted Extra attention is needed

Comments

@alamb
Copy link
Contributor

alamb commented Apr 26, 2021

Note: migrated from original JIRA: https://issues.apache.org/jira/browse/ARROW-11020

Implement better tests for ParquetExec
@alamb alamb added the datafusion Changes in the datafusion crate label Apr 26, 2021
@alamb
Copy link
Contributor Author

alamb commented Apr 26, 2021

Comment from Krisztian Szucs(kszucs) @ 2021-01-12T14:48:02.701+0000:

Postponing to 4.0

@houqp houqp added good first issue Good for newcomers help wanted Extra attention is needed labels Sep 15, 2021
@houqp
Copy link
Member

houqp commented Sep 15, 2021

looks like this could potentially be a good first issue, but we need to provide more details and define what better means here :)

@novemberkilo
Copy link

@alamb @houqp I am interested in working on this once you've helped specify (it a bit) better :)

@alamb
Copy link
Contributor Author

alamb commented Nov 29, 2021

I think that since @andygrove filed https://issues.apache.org/jira/browse/ARROW-11020 this one he would probably need to weigh in on what he had in mind.

The ParquetExec implementation has changed fairly significantly in the last few months (e.g. #1141) as well; Perhaps @rdettai has some ideas of areas that need better test coverage

@alamb
Copy link
Contributor Author

alamb commented Sep 18, 2024

ParquetExec is pretty well tested in my mind now -- I don't think this ticket is actionable

@alamb alamb closed this as completed Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
datafusion Changes in the datafusion crate good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@novemberkilo @alamb @houqp