clintropolis opened a new pull request, #19398:
URL: https://github.com/apache/druid/pull/19398

   ### Description
   
   changes:
   * add nullable `minTime`/`maxTime` Long fields to `ProjectionMetadata` 
alongside the existing `numRows`
   * track min/max __time in `IndexMergerBase.mergeIndexesAndWriteColumns` via 
`timestampSelector.getLong()` on each row; surface via `IndexMergeResult` (null 
when zero rows are walked)
   * `IndexMergerV10` wires the merge result's values into the base-table 
`ProjectionMetadata`; aggregate projections leave the new fields null for now, 
though we can easily add this in future work if useful
   * tracking is done regardless of segment sort order, non-time-sorted 
segments (`DimensionsSpec.forceSegmentSortByTime = false`) also store these 
values accurately (just with less utility since we can't use the information to 
skip rows, though we could use them to know if there are no rows within the 
time range)
   * write-side only: fields are persisted but not yet consumed. A follow-up 
will add a partial V10-aware `TimeBoundaryInspector` that reads them


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to