diff options
| author | Liwei Lin <lwlin7@gmail.com> | 2016-07-24 08:35:57 +0100 |
|---|---|---|
| committer | Sean Owen <sowen@cloudera.com> | 2016-07-24 08:35:57 +0100 |
| commit | d6795c7a254b83d4ae4785f3add74981e5273c91 (patch) | |
| tree | b705111bb4f48adad23a1cee7743b8cc53b83816 /python | |
| parent | e3c7039b45763ca70c91432e7bdb8494ab010e62 (diff) | |
| download | spark-d6795c7a254b83d4ae4785f3add74981e5273c91.tar.gz spark-d6795c7a254b83d4ae4785f3add74981e5273c91.tar.bz2 spark-d6795c7a254b83d4ae4785f3add74981e5273c91.zip | |
[SPARK-16515][SQL][FOLLOW-UP] Fix test `script` on OS X/Windows...
## Problem
The current `sed` in `test_script.sh` is missing a `$`, leading to the failure of `script` test on OS X:
```
== Results ==
!== Correct Answer - 2 == == Spark Answer - 2 ==
![x1_y1] [x1]
![x2_y2] [x2]
```
In addition, this `script` test would also fail on systems like Windows where we couldn't be able to invoke `bash` or `echo | sed`.
## What changes were proposed in this pull request?
This patch
- fixes `sed` in `test_script.sh`
- adds command guards so that the `script` test would pass on systems like Windows
## How was this patch tested?
- Jenkins
- Manually verified tests pass on OS X
Author: Liwei Lin <lwlin7@gmail.com>
Closes #14280 from lw-lin/osx-sed.
Diffstat (limited to 'python')
0 files changed, 0 insertions, 0 deletions