Skip to content

Test column sorting in benchmarks#136

Merged
marcin-krystianc merged 9 commits intoG-Research:masterfrom
marcin-krystianc:dev-20260430-benchmarks
May 5, 2026
Merged

Test column sorting in benchmarks#136
marcin-krystianc merged 9 commits intoG-Research:masterfrom
marcin-krystianc:dev-20260430-benchmarks

Conversation

@marcin-krystianc
Copy link
Copy Markdown
Contributor

  • Reading columns in sorted order may have a positive effect on file reading performance
  • When testing prefetch_page_cache, we need to pass a reasonable range_size_limit.

@marcin-krystianc marcin-krystianc requested a review from adamreeve May 5, 2026 08:45
@marcin-krystianc marcin-krystianc marked this pull request as ready for review May 5, 2026 08:45
Copy link
Copy Markdown
Contributor

@adamreeve adamreeve left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me thanks Marcin, just a minor formatting issue with the readme.

Comment thread README.md
Comment thread README.md Outdated
marcin-krystianc and others added 3 commits May 5, 2026 12:08
Co-authored-by: Adam Reeve <adreeve@gmail.com>
@marcin-krystianc marcin-krystianc requested a review from adamreeve May 5, 2026 10:11
@marcin-krystianc
Copy link
Copy Markdown
Contributor Author

Looks good to me thanks Marcin, just a minor formatting issue with the readme.

Fixed

@marcin-krystianc marcin-krystianc merged commit d11f9f1 into G-Research:master May 5, 2026
23 checks passed
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