Skip to content

ARROW-10647: Add in Rust performance testing file#15

Closed
alamb wants to merge 2 commits into
apache:masterfrom
alamb:alamb/ARROW-10647-rust-parquet-benchmarks
Closed

ARROW-10647: Add in Rust performance testing file#15
alamb wants to merge 2 commits into
apache:masterfrom
alamb:alamb/ARROW-10647-rust-parquet-benchmarks

Conversation

@alamb

@alamb alamb commented Nov 18, 2020

Copy link
Copy Markdown
Contributor

We are adding a rust performance benchmark to the arrow repo, in apache/arrow#8708

In order to avoid adding 653K of binary data to that repo, @sunchao suggested adding it here

@alamb

alamb commented Nov 18, 2020

Copy link
Copy Markdown
Contributor Author

I am not sure how to request a review on this PR (github does not seem to want to let me do so). FYI @wesm

@wesm

wesm commented Nov 19, 2020

Copy link
Copy Markdown
Member

Hm. Is there a plan for "endogenous" performance testing (creating test files on the fly)?

@wesm

wesm commented Nov 19, 2020

Copy link
Copy Markdown
Member

(you have to be a "Parquet" committer to be able to merge patches here or request reviews)

@alamb

alamb commented Nov 19, 2020

Copy link
Copy Markdown
Contributor Author

@wesm

Hm. Is there a plan for "endogenous" performance testing (creating test files on the fly)?

Not that I am aware of. Given what these files are being used for, I don't see any reason we can't generate them on the fly. I'll work with @sunchao on that.

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 this pull request may close these issues.

2 participants