[arrow] branch master updated: ARROW-5270: [C++] reduce json-reader-test's working size

[ap...@apache.org]

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

apitrou 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 0882924  ARROW-5270: [C++] reduce json-reader-test's working size
0882924 is described below

commit 08829248fd540b7e3bd96b980e357f8a4db7970e
Author: Benjamin Kietzman <be...@gmail.com>
AuthorDate: Tue May 7 14:11:18 2019 +0200

    ARROW-5270: [C++] reduce json-reader-test's working size
    
    Valgrind on Travis is slow due to this test, which doesn't need to be so big. @pitrou
    
    Author: Benjamin Kietzman <be...@gmail.com>
    
    Closes #4264 from bkietz/5270-Reenable-Valgrind-on-Travis-CI and squashes the following commits:
    
    61d72beb6 <Benjamin Kietzman> reduce json-reader-test's working size
---
 cpp/src/arrow/json/reader-test.cc | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/cpp/src/arrow/json/reader-test.cc b/cpp/src/arrow/json/reader-test.cc
index aefd25a..f538ce7 100644
--- a/cpp/src/arrow/json/reader-test.cc
+++ b/cpp/src/arrow/json/reader-test.cc
@@ -195,7 +195,7 @@ std::string RowsOfOneColumn(string_view name, std::initializer_list<T> values,
 }
 
 TEST(ReaderTest, MultipleChunksParallel) {
-  int64_t count = 1 << 20;
+  int64_t count = 1 << 10;
 
   ParseOptions parse_options;
   parse_options.unexpected_field_behavior = UnexpectedFieldBehavior::InferType;
@@ -233,9 +233,6 @@ TEST(ReaderTest, MultipleChunksParallel) {
     }
   }
 
-  // std::cout << serial->column(0)->data()->num_chunks() << " chunks, " << json.size()
-  //           << " bytes" << std::endl;
-
   AssertTablesEqual(*serial, *threaded);
 }