imay opened a new issue #1386: Add new SegmentGroupIterator URL: https://github.com/apache/incubator-doris/issues/1386 Now we have SegmentGroup who have some segments, and these segment is ordered. And we have another class named Rowset who is container of SegmentGroup. But I think we can merge SegmentGroup and Rowset into one class who is container of segment. And the difference is that these segments is not ordered. We can use flag or something to express it. But in this refactor we should keep these two class for make things easy. so we need to create a SegmentGroupIterator to read data in segment group. This class will create one SegmentIterator for each segment as its children. And because segments are ordered, this class will just union all children's output which will be very easy. #1305
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
