Skip to content

Conversation

@silverweed
Copy link
Contributor

@silverweed silverweed commented Oct 30, 2025

Adds a test to the merger where we merge 3 RNTuples with different schemas, the first 2 of which have 0 entries.
Verify that it behaves as expected (Filter and Strict fail, Union produces an output with the union of the schemas and non-deferred entries from the 3rd RNTuple)

Checklist:

  • tested changes locally
  • updated the docs (if necessary)

Copy link
Contributor

@jblomer jblomer left a comment

Choose a reason for hiding this comment

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

Nice! Perhaps extend the test to make the first two ntuples zero-sized.

@silverweed silverweed force-pushed the ntuple_merge_first_noentries_test branch from c78a58d to 94ca0ae Compare October 30, 2025 10:03
@github-actions
Copy link

Test Results

    22 files      22 suites   3d 15h 1m 26s ⏱️
 3 705 tests  3 703 ✅ 0 💤 2 ❌
79 566 runs  79 564 ✅ 0 💤 2 ❌

For more details on these failures, see this check.

Results for commit 94ca0ae.

@silverweed silverweed merged commit af1e4ce into root-project:master Nov 5, 2025
25 of 27 checks passed
@silverweed silverweed deleted the ntuple_merge_first_noentries_test branch November 5, 2025 14:01
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.

2 participants