Yingchun Lai has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/20287 )
Change subject: [common] fix compilation warning
......................................................................
[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]>
---
M src/kudu/common/partition_pruner.cc
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Alexey Serbin: Verified
Mahesh Reddy: Looks good to me, but someone else must approve
Yifan Zhang: Looks good to me, approved
--
To view, visit http://gerrit.cloudera.org:8080/20287
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I4bc8cd869988f69c35a98872cefc072321d6ef3f
Gerrit-Change-Number: 20287
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <[email protected]>
Gerrit-Reviewer: Yifan Zhang <[email protected]>
Gerrit-Reviewer: Yingchun Lai <[email protected]>
