-
Notifications
You must be signed in to change notification settings - Fork 32
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
Hook on-disk vortex files into benchmarking #565
Conversation
cf43e18
to
f4faa93
Compare
dac788d
to
6709da8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's revert the visibility modifications
EquivalenceProperties::new(file_scan_config.file_schema.clone()), | ||
partitioning, | ||
EquivalenceProperties::new(projected_schema), | ||
Partitioning::UnknownPartitioning(1), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should figure this one out. Requires some extra config in the reader
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right now, we do only use a single partition in the file scan as all files are in the same group.
Adding the Vortex on-file format to our benchmarking suite