Github user anantasty commented on the pull request:
https://github.com/apache/spark/pull/941#issuecomment-45018927
That might actually be a smarter idea. We could just extend subprocess to
add check_all and use that.
On Tue, Jun 3, 2014 at 2:43 PM, Patrick Wendell <[email protected]>
wrote:
> Hey one of they things about check_call is that it would give throw an
> exception if the suprocess returned nonzero. Does this have the same
> behavior? This is actually is somewhat important because otherwise the
> script will happily continue having captured the wrong output and it will
> downstream throw an error that is really hard to debug.
>
> I wonder if we should just inline the definition of check_output from
> Python 7. That's what's suggested here:
>
>
>
http://stackoverflow.com/questions/4814970/subprocess-check-output-doesnt-seem-to-exist-python-2-6-5
>
> â
> Reply to this email directly or view it on GitHub
> <https://github.com/apache/spark/pull/941#issuecomment-45017653>.
>
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
