gongxun0928 commented on code in PR #1324:
URL: https://github.com/apache/cloudberry/pull/1324#discussion_r2297005813
##########
contrib/pax_storage/src/cpp/access/pax_access_handle.cc:
##########
@@ -453,6 +453,7 @@ uint32 PaxAccessMethod::ScanFlags(Relation relation) {
flags |= SCAN_FORCE_BIG_WRITE_LOCK;
#endif
+ flags |= SCAN_SUPPORT_RUNTIME_FILTER;
Review Comment:
When gp_enable_runtime_filter_pushdown is enabled by default, it applies to
all queries on PAX-formatted tables, which is expected to affect TPC-DS/TPC-H
performance. IN and min-max filters rely on column-level statistics, whereas
bloom filters perform row-level filtering. Even without IN or min-max
filtering, simply pushing the bloom filter from SeqNext() down to the TableAM
level can reduce unnecessary memory copies. For example, if a row has already
been filtered by a bloom filter expression, there is no need to construct and
return the corresponding TableTupleSlot for that row.
--
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: commits-unsubscr...@cloudberry.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cloudberry.apache.org
For additional commands, e-mail: commits-h...@cloudberry.apache.org
