diff options
author | Josh Rosen <joshrosen@databricks.com> | 2016-06-16 14:18:58 -0700 |
---|---|---|
committer | Andrew Or <andrew@databricks.com> | 2016-06-16 14:18:58 -0700 |
commit | acef843f67e770f0a2709fb3fbd1a53c200b2bc5 (patch) | |
tree | c976ba421b7bbd85f6281dda6c6e8f221dd3bd7b /sql | |
parent | bbad4cb48df2ac3ed7edb4c02db79540bd4085d8 (diff) | |
download | spark-acef843f67e770f0a2709fb3fbd1a53c200b2bc5.tar.gz spark-acef843f67e770f0a2709fb3fbd1a53c200b2bc5.tar.bz2 spark-acef843f67e770f0a2709fb3fbd1a53c200b2bc5.zip |
[SPARK-15975] Fix improper Popen retcode code handling in dev/run-tests
In the `dev/run-tests.py` script we check a `Popen.retcode` for success using `retcode > 0`, but this is subtlety wrong because Popen's return code will be negative if the child process was terminated by a signal: https://docs.python.org/2/library/subprocess.html#subprocess.Popen.returncode
In order to properly handle signals, we should change this to check `retcode != 0` instead.
Author: Josh Rosen <joshrosen@databricks.com>
Closes #13692 from JoshRosen/dev-run-tests-return-code-handling.
Diffstat (limited to 'sql')
0 files changed, 0 insertions, 0 deletions