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.

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

[ntuple] Fix-up read options #17574

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jblomer
Copy link
Contributor

@jblomer jblomer commented Jan 30, 2025

Preparation for moving read options out of experimental.

This option has been useful for performance studies. It specifies the
number of clusters to be preloaded in the cluster cache. Since the size
of clusters is a priori unknown, this option is not well suited as a
general interface. Eventually, the option will be replaced by a prefetch
setting based on available memory ("prefill at max x MB").
Copy link

github-actions bot commented Jan 30, 2025

Test Results

    18 files      18 suites   4d 3h 34m 2s ⏱️
 2 683 tests  2 681 ✅ 0 💤 2 ❌
46 598 runs  46 596 ✅ 0 💤 2 ❌

For more details on these failures, see this check.

Results for commit 22fb187.

♻️ This comment has been updated with latest results.

@jblomer jblomer force-pushed the ntuple-fix-read-options branch from 306b4b5 to 22fb187 Compare January 30, 2025 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants