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]));