You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@parquet.apache.org by uw...@apache.org on 2018/03/28 16:19:18 UTC

[parquet-cpp] branch master updated: PARQUET-1071: Check that arrow::FileWriter::Close() is idempotent

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

uwe pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/parquet-cpp.git


The following commit(s) were added to refs/heads/master by this push:
     new 72063f3  PARQUET-1071: Check that arrow::FileWriter::Close() is idempotent
72063f3 is described below

commit 72063f3afa76caeb1785143889928c0c7312e677
Author: Antoine Pitrou <an...@python.org>
AuthorDate: Wed Mar 28 18:18:33 2018 +0200

    PARQUET-1071: Check that arrow::FileWriter::Close() is idempotent
    
    Author: Antoine Pitrou <an...@python.org>
    
    Closes #449 from pitrou/PARQUET-1071-arrow-filewriter-close-idempotent and squashes the following commits:
    
    8c3fb19 [Antoine Pitrou] PARQUET-1071: Check that arrow::FileWriter::Close() is idempotent
---
 src/parquet/arrow/arrow-reader-writer-test.cc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/parquet/arrow/arrow-reader-writer-test.cc b/src/parquet/arrow/arrow-reader-writer-test.cc
index 8051ff1..46218be 100644
--- a/src/parquet/arrow/arrow-reader-writer-test.cc
+++ b/src/parquet/arrow/arrow-reader-writer-test.cc
@@ -574,6 +574,8 @@ class TestParquetIO : public ::testing::Test {
     ASSERT_OK_NO_THROW(writer.NewRowGroup(values->length()));
     ASSERT_OK_NO_THROW(writer.WriteColumnChunk(*values));
     ASSERT_OK_NO_THROW(writer.Close());
+    // writer.Close() should be idempotent
+    ASSERT_OK_NO_THROW(writer.Close());
   }
 
   std::shared_ptr<InMemoryOutputStream> sink_;

-- 
To stop receiving notification emails like this one, please contact
uwe@apache.org.