gaodayue opened a new pull request #1698: refactor RowsetReaderContext URL: https://github.com/apache/incubator-doris/pull/1698 This PR refactors RowsetReaderContext and adds more comments to help developers understand the read path. Tha main changes are - move index stream cache stuff (`is_using_cache` and `lru_cache`) out of RowsetReaderContext because that's specific to alpha rowset and whether to use that cache can be determined solely by `reader_type` - `preaggregation` is replaced with `need_ordered_result` to improve code understandability and to simplify RowsetReader implementations
---------------------------------------------------------------- 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]
