You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ks...@apache.org on 2020/04/11 10:35:47 UTC

[arrow] branch master updated: ARROW-8397: [C++] Fail to compile aggregate_test.cc on Ubuntu 16.04

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 10c719c  ARROW-8397: [C++] Fail to compile aggregate_test.cc on Ubuntu 16.04
10c719c is described below

commit 10c719c7241c1ea2834902a2639ff6bd3dd55966
Author: Krisztián Szűcs <sz...@gmail.com>
AuthorDate: Sat Apr 11 12:35:27 2020 +0200

    ARROW-8397: [C++] Fail to compile aggregate_test.cc on Ubuntu 16.04
    
    Closes #6895 from kszucs/ce
    
    Authored-by: Krisztián Szűcs <sz...@gmail.com>
    Signed-off-by: Krisztián Szűcs <sz...@gmail.com>
---
 cpp/src/arrow/compute/kernels/aggregate_test.cc | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/cpp/src/arrow/compute/kernels/aggregate_test.cc b/cpp/src/arrow/compute/kernels/aggregate_test.cc
index bb48861..444bda6 100644
--- a/cpp/src/arrow/compute/kernels/aggregate_test.cc
+++ b/cpp/src/arrow/compute/kernels/aggregate_test.cc
@@ -398,9 +398,9 @@ class TestNumericMinMaxKernel : public ComputeFixture, public TestBase {
     ASSERT_OK(MinMax(&this->ctx_, options, array, &out));
 
     ASSERT_TRUE(out.is_collection());
-    for (const auto& out : out.collection()) {
-      ASSERT_TRUE(out.is_scalar());
-      ASSERT_FALSE(out.scalar()->is_valid);
+    for (const auto& item : out.collection()) {
+      ASSERT_TRUE(item.is_scalar());
+      ASSERT_FALSE(item.scalar()->is_valid);
     }
   }