GitHub user dilipbiswal opened a pull request:
https://github.com/apache/spark/pull/22000
[SPARK-25025][SQL] Remove the default value of isAll in INTERSECT/EXCEPT
## What changes were proposed in this pull request?
Having the default value of isAll in the logical plan nodes
INTERSECT/EXCEPT could introduce bugs when the callers are not aware of it.
This PR removes the default value and makes caller explicitly specify them.
## How was this patch tested?
This is a refactoring changes. Existing tests test the functionality
already.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dilipbiswal/spark SPARK-25025
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/22000.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 #22000
----
commit 57650985eca7de23c6c8cf2812d673702c8e1f3d
Author: Dilip Biswal <dbiswal@...>
Date: 2018-08-04T21:32:57Z
[SPARK-25025] Remove the default value of isAll in INTERSECT/EXCEPT
----
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
