You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by da...@apache.org on 2022/10/11 14:26:04 UTC

[doris] branch branch-1.1-lts updated: [fix](bloom filter) wrong result of bloom filter for is not null (#13299)

This is an automated email from the ASF dual-hosted git repository.

dataroaring pushed a commit to branch branch-1.1-lts
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/branch-1.1-lts by this push:
     new c8b515e703 [fix](bloom filter) wrong result of bloom filter for is not null (#13299)
c8b515e703 is described below

commit c8b515e7032c646a76a49d98e077468396202809
Author: TengJianPing <18...@users.noreply.github.com>
AuthorDate: Tue Oct 11 22:25:58 2022 +0800

    [fix](bloom filter) wrong result of bloom filter for is not null (#13299)
---
 be/src/olap/olap_cond.h | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/be/src/olap/olap_cond.h b/be/src/olap/olap_cond.h
index 6146859d00..f1a982c114 100644
--- a/be/src/olap/olap_cond.h
+++ b/be/src/olap/olap_cond.h
@@ -82,7 +82,10 @@ public:
     bool eval(const BloomFilter& bf) const;
     bool eval(const segment_v2::BloomFilter* bf) const;
 
-    bool can_do_bloom_filter() const { return op == OP_EQ || op == OP_IN || op == OP_IS; }
+    bool can_do_bloom_filter() const {
+        return op == OP_EQ || op == OP_IN
+            || (op == OP_IS && operand_field->is_null());
+    }
 
     CondOp op = OP_NULL;
     // valid when op is not OP_IN and OP_NOT_IN


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org