kangpinghuang commented on a change in pull request #1633: add zone map
page(#1390)
URL: https://github.com/apache/incubator-doris/pull/1633#discussion_r315021694
##########
File path: be/src/olap/iterators.h
##########
@@ -35,15 +36,19 @@ struct StorageReadOptions {
// If include_lower_bound is true, data equal with lower_bound will
// be read
- bool include_lower_bound;
+ bool include_lower_bound = false;
// upper_bound defines the extend upto which the iterator can return
// data.
std::shared_ptr<RowCursor> upper_bound;
// If include_upper_bound is true, data equal with upper_bound will
// be read
- bool include_upper_bound;
+ bool include_upper_bound = false;
+
+ // reader's column predicates
+ // used to filter data by zone map/bloom filter/second index
+ std::shared_ptr<Conditions> conditions;
Review comment:
it is convenient for client to use by using shared_ptr. the client can use
the Options without caring about the lifecycle of the pointer.
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@doris.apache.org
For additional commands, e-mail: dev-h...@doris.apache.org
