GitHub user viirya opened a pull request:
https://github.com/apache/spark/pull/19161
[SPARK-21726][SQL][Follow-UP] Check for structural integrity of the plan in
Optimzer in test mode
## What changes were proposed in this pull request?
The condition in `Optimizer.isPlanIntegral` is wrong. We should always
return `true` if not in test mode.
## How was this patch tested?
N/A.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/viirya/spark-1 SPARK-21726-followup
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/19161.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #19161
----
commit cf76ca57920c92c35f82caf2752894536f0617e2
Author: Liang-Chi Hsieh <vii...@gmail.com>
Date: 2017-09-08T07:16:11Z
Fix the condition.
----
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
