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

laiyingchun pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git


The following commit(s) were added to refs/heads/master by this push:
     new 2596a515c [common] fix compilation warning
2596a515c is described below

commit 2596a515cfafc247e97b6052d9139ff8643fed22
Author: Alexey Serbin <[email protected]>
AuthorDate: Mon Jul 31 10:20:24 2023 -0700

    [common] fix compilation warning
    
    This patch fixes compilation warning in partition_pruner.cc:
    
      partition_pruner.cc:222:18: warning: statement has no effect 
[-Wunused-value]
         DCHECK_NOTNULL(predicate_values_selected);
                        ^
      partition_pruner.cc:223:18: warning: statement has no effect 
[-Wunused-value]
         DCHECK_NOTNULL(hash_bucket_bitset);
                      ^
    This is a follow-up to 5d6774b1022f26d33abd6cc9fb99507490849428.
    
    This patch doesn't contain any functional modifications.
    
    Change-Id: I4bc8cd869988f69c35a98872cefc072321d6ef3f
    Reviewed-on: http://gerrit.cloudera.org:8080/20287
    Tested-by: Alexey Serbin <[email protected]>
    Reviewed-by: Mahesh Reddy <[email protected]>
    Reviewed-by: Yifan Zhang <[email protected]>
---
 src/kudu/common/partition_pruner.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/kudu/common/partition_pruner.cc 
b/src/kudu/common/partition_pruner.cc
index 7d954be8e..a5bb56cb9 100644
--- a/src/kudu/common/partition_pruner.cc
+++ b/src/kudu/common/partition_pruner.cc
@@ -219,8 +219,8 @@ void PartitionPruner::ComputeHashBuckets(const Schema& 
schema, // NOLINT(misc-no
                                          const vector<vector<const void*>>& 
predicate_values_list,
                                          vector<const void*>* 
predicate_values_selected,
                                          vector<bool>* hash_bucket_bitset) {
-  DCHECK_NOTNULL(predicate_values_selected);
-  DCHECK_NOTNULL(hash_bucket_bitset);
+  DCHECK(predicate_values_selected);
+  DCHECK(hash_bucket_bitset);
   bool all_hash_bucket_needed = true;
   for (const auto b : *hash_bucket_bitset) {
     all_hash_bucket_needed &= b;

Reply via email to