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


The following commit(s) were added to refs/heads/master by this push:
     new 7354a19  ARROW-2284: [Python] Fix error display on test_plasma error
7354a19 is described below

commit 7354a190d78a0cadb7e0db9a2014b8a5e2848410
Author: Antoine Pitrou <[email protected]>
AuthorDate: Fri Mar 9 10:19:44 2018 -0500

    ARROW-2284: [Python] Fix error display on test_plasma error
    
    Just a trivial fix.  stderr is captured by py.test, not by the subprocess 
call.
    
    Author: Antoine Pitrou <[email protected]>
    
    Closes #1724 from pitrou/ARROW-2284 and squashes the following commits:
    
    46a692c9 <Antoine Pitrou> ARROW-2284:  Fix error display on test_plasma 
error
---
 python/pyarrow/tests/test_plasma.py | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/python/pyarrow/tests/test_plasma.py 
b/python/pyarrow/tests/test_plasma.py
index b4e8649..1df213d 100644
--- a/python/pyarrow/tests/test_plasma.py
+++ b/python/pyarrow/tests/test_plasma.py
@@ -165,10 +165,8 @@ def 
start_plasma_store(plasma_store_memory=DEFAULT_PLASMA_STORE_MEMORY,
             time.sleep(0.1)
         rc = proc.poll()
         if rc is not None:
-            err = proc.stderr.read().decode()
             raise RuntimeError("plasma_store exited unexpectedly with "
-                               "code %d. Error output follows:\n%s\n"
-                               % (rc, err))
+                               "code %d" % (rc,))
 
         yield plasma_store_name, proc
     finally:

-- 
To stop receiving notification emails like this one, please contact
[email protected].

Reply via email to