You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@doris.apache.org by GitBox <gi...@apache.org> on 2019/08/25 09:15:40 UTC
[GitHub] [incubator-doris] gaodayue opened a new pull request #1698:
refactor RowsetReaderContext
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:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@doris.apache.org
For additional commands, e-mail: dev-help@doris.apache.org