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

   ### Description
   Small change to split out a new `PhysicalSegmentColumnInspector` from 
`PhysicalSegmentInspector` that takes all of the column specific methods (and 
the `ColumnInspector` part) to assist the partial load functionality I have 
been working on, since the methods I have moved to 
`PhysicalSegmentColumnInspector` currently at least need the actual column 
present to correctly implement, however the remaining methods in 
`PhysicalSegmentInspector` can be answered purely with metadata. 
`PhysicalSegmentColumnInspector` is only used by `SegmentAnalyzer` for segment 
metadata queries for column analysis, so I think this change shouldn't be very 
disruptive, as the wider used methods like row count and metadata retrieval 
remain on the existing interface.


-- 
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