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;