Skip to content

Commit

Permalink
[ntuple][NFC] Clarify deferred column documentation
Browse files Browse the repository at this point in the history
  • Loading branch information
enirolf committed Jun 11, 2024
1 parent 1e0b7f9 commit 2ae6678
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tree/ntuple/v7/src/RNTupleDescriptor.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -610,10 +610,10 @@ ROOT::Experimental::Internal::RClusterDescriptorBuilder::AddExtendedColumnRanges

pageRange.fPhysicalColumnId = physicalId;
}
// Fixup the RColumnRange and RPageRange in deferred columns. We know what the first element index and
// Fixup the RColumnRange and RPageRange in deferred columns. We know what the first element index and
// number of elements should have been if the column was not deferred; fix those and let
// `ExtendToFitColumnRange()` synthesize RPageInfos accordingly.
// Note that a column whose first element index is != 0 already met the criteria of
// Note that a deferred column (i.e, whose first element index is > 0) already met the criteria of
// `RFieldBase::EntryToColumnElementIndex()`, i.e. it is a principal column reachable from the field zero
// excluding subfields of collection and variant fields.
if (c.IsDeferredColumn()) {
Expand Down

0 comments on commit 2ae6678

Please sign in to comment.