You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by al...@apache.org on 2020/11/23 02:25:27 UTC
[kudu] branch master updated: [test] fix compilation warning
This is an automated email from the ASF dual-hosted git repository.
alexey 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 0faa2a6 [test] fix compilation warning
0faa2a6 is described below
commit 0faa2a6fbea19ae91e5bd9fe0f6c16d58137e475
Author: Alexey Serbin <al...@apache.org>
AuthorDate: Sat Nov 21 23:05:03 2020 -0800
[test] fix compilation warning
src/kudu/integration-tests/flex_partitioning-itest.cc:221:32: warning: loop variable 'split' of type 'const vector<int32_t>' (aka 'const vector<int>') creates a copy from type 'const vector<int32_t>' [-Wrange-loop-construct]
for (const vector<int32_t> split : range_partition.splits) {
^
Change-Id: I16ab253ab99808c2a69b0520a3a6b6253f27f2f8
Reviewed-on: http://gerrit.cloudera.org:8080/16766
Reviewed-by: Grant Henke <gr...@apache.org>
Tested-by: Grant Henke <gr...@apache.org>
---
src/kudu/integration-tests/flex_partitioning-itest.cc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/kudu/integration-tests/flex_partitioning-itest.cc b/src/kudu/integration-tests/flex_partitioning-itest.cc
index 4f6cca2..6da3b1f 100644
--- a/src/kudu/integration-tests/flex_partitioning-itest.cc
+++ b/src/kudu/integration-tests/flex_partitioning-itest.cc
@@ -218,7 +218,7 @@ class FlexPartitioningITest : public KuduTest,
}
vector<const KuduPartialRow*> split_rows;
- for (const vector<int32_t> split : range_partition.splits) {
+ for (const auto& split : range_partition.splits) {
KuduPartialRow* row = schema.NewRow();
for (int i = 0; i < split.size(); i++) {
ASSERT_OK(row->SetInt32(range_partition.columns[i], split[i]));