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] split loading and processing meta-data #15909

Merged
merged 1 commit into from
Jun 26, 2024

Conversation

jblomer
Copy link
Contributor

@jblomer jblomer commented Jun 22, 2024

Adds RPageSource::LoadStructure[Impl](). This allows to load header and footer without processing them. Decompression and deserialization will still happen in Attach(), which calls LoadStructure if it hasn't been called before.

This will facilitate asynchronous open of files for RDF MT scheduling.

Adds RPageSource::LoadStructure[Impl](). This allows to load header and
footer without processing them. Decompression and deserialization will
still happen in Attach(), which calls LoadStructure if it hasn't been
called before.

This will facilitate asynchronous open of files for RDF MT scheduling.
Copy link

Test Results

    13 files      13 suites   3d 4h 13m 35s ⏱️
 2 646 tests  2 646 ✅ 0 💤 0 ❌
32 587 runs  32 587 ✅ 0 💤 0 ❌

Results for commit b1f7e20.

Copy link
Member

@hahnjo hahnjo left a comment

Choose a reason for hiding this comment

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

LGTM

@jblomer jblomer merged commit 722eb46 into root-project:master Jun 26, 2024
17 of 18 checks passed
@jblomer jblomer deleted the ntuple-load-metadata branch June 26, 2024 13:25
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.

None yet

2 participants