ARROW-200: [C++/Python] Return error status on string initialization …

…failure Author: Micah Kornfield <emkornfield@gmail.com> Closes #88 from emkornfield/emk_arrow_200 and squashes the following commits: 37e23be [Micah Kornfield] ARROW-200: Return error status on string initialization failure
apache · Jun 8, 2016 · bc6c4c8 · bc6c4c8
1 parent 9ce13a0
commit bc6c4c8
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/python/src/pyarrow/adapters/pandas.cc b/python/src/pyarrow/adapters/pandas.cc
@@ -669,7 +669,7 @@ class ArrowDeserializer {
 
           out_values[i] = make_pystring(data, length);
           if (out_values[i] == nullptr) {
-            return Status::OK();
+            return Status::UnknownError("String initialization failed");
           }
         }
       }
@@ -678,7 +678,7 @@ class ArrowDeserializer {
         data = string_arr->GetValue(i, &length);
         out_values[i] = make_pystring(data, length);
         if (out_values[i] == nullptr) {
-          return Status::OK();
+          return Status::UnknownError("String initialization failed");
         }
       }
     }