You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by we...@apache.org on 2018/09/27 12:19:55 UTC

[arrow] 16/24: PARQUET-1053: Fix unused result warnings due to unchecked Statuses

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

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

commit 09cd54540c0b86323de4ccc2a5e918009845677b
Author: Phillip Cloud <cp...@gmail.com>
AuthorDate: Mon Jul 10 22:03:25 2017 -0400

    PARQUET-1053: Fix unused result warnings due to unchecked Statuses
    
    Author: Phillip Cloud <cp...@gmail.com>
    
    Closes #369 from cpcloud/PARQUET-1053 and squashes the following commits:
    
    e0598b4 [Phillip Cloud] PARQUET-1053: Fix unused result warnings due to unchecked Statuses
    
    Change-Id: I91d267f56685c0e1267b6069c5bff52f2a1eca15
---
 cpp/examples/parquet/reader-writer.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/cpp/examples/parquet/reader-writer.cc b/cpp/examples/parquet/reader-writer.cc
index 6f21f6c..210968c 100644
--- a/cpp/examples/parquet/reader-writer.cc
+++ b/cpp/examples/parquet/reader-writer.cc
@@ -22,6 +22,7 @@
 #include <memory>
 
 #include <arrow/io/file.h>
+#include <arrow/util/logging.h>
 
 #include <parquet/api/reader.h>
 #include <parquet/api/writer.h>
@@ -216,7 +217,7 @@ int main(int argc, char** argv) {
     file_writer->Close();
 
     // Write the bytes to file
-    out_file->Close();
+    DCHECK(out_file->Close().ok());
   } catch (const std::exception& e) {
     std::cerr << "Parquet write error: " << e.what() << std::endl;
     return -1;