Repository: spark
Updated Branches:
  refs/heads/branch-1.6 865dd8bcc -> b8da77ef7


[SPARK-12520] [PYSPARK] Correct Descriptions and Add Use Cases in Equi-Join

After reading the JIRA https://issues.apache.org/jira/browse/SPARK-12520, I 
double checked the code.

For example, users can do the Equi-Join like
  ```df.join(df2, 'name', 'outer').select('name', 'height').collect()```
- There exists a bug in 1.5 and 1.4. The code just ignores the third parameter 
(join type) users pass. However, the join type we called is `Inner`, even if 
the user-specified type is the other type (e.g., `Outer`).
- After a PR: https://github.com/apache/spark/pull/8600, the 1.6 does not have 
such an issue, but the description has not been updated.

Plan to submit another PR to fix 1.5 and issue an error message if users 
specify a non-inner join type when using Equi-Join.

Author: gatorsmile <gatorsm...@gmail.com>

Closes #10477 from gatorsmile/pyOuterJoin.


Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/b8da77ef
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/b8da77ef
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/b8da77ef

Branch: refs/heads/branch-1.6
Commit: b8da77ef776ab9cdc130a70293d75e7bdcdf95b0
Parents: 865dd8b
Author: gatorsmile <gatorsm...@gmail.com>
Authored: Sun Dec 27 23:18:48 2015 -0800
Committer: Davies Liu <davies....@gmail.com>
Committed: Sun Dec 27 23:19:50 2015 -0800

----------------------------------------------------------------------
 python/pyspark/sql/dataframe.py | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/b8da77ef/python/pyspark/sql/dataframe.py
----------------------------------------------------------------------
diff --git a/python/pyspark/sql/dataframe.py b/python/pyspark/sql/dataframe.py
index b15b8d7..a0fdaf3 100644
--- a/python/pyspark/sql/dataframe.py
+++ b/python/pyspark/sql/dataframe.py
@@ -608,13 +608,16 @@ class DataFrame(object):
         :param on: a string for join column name, a list of column names,
             , a join expression (Column) or a list of Columns.
             If `on` is a string or a list of string indicating the name of the 
join column(s),
-            the column(s) must exist on both sides, and this performs an inner 
equi-join.
+            the column(s) must exist on both sides, and this performs an 
equi-join.
         :param how: str, default 'inner'.
             One of `inner`, `outer`, `left_outer`, `right_outer`, `leftsemi`.
 
         >>> df.join(df2, df.name == df2.name, 'outer').select(df.name, 
df2.height).collect()
         [Row(name=None, height=80), Row(name=u'Alice', height=None), 
Row(name=u'Bob', height=85)]
 
+        >>> df.join(df2, 'name', 'outer').select('name', 'height').collect()
+        [Row(name=u'Tom', height=80), Row(name=u'Alice', height=None), 
Row(name=u'Bob', height=85)]
+
         >>> cond = [df.name == df3.name, df.age == df3.age]
         >>> df.join(df3, cond, 'outer').select(df.name, df3.age).collect()
         [Row(name=u'Bob', age=5), Row(name=u'Alice', age=2)]


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to