Skip to content

Conversation

@zjffdu
Copy link
Contributor

@zjffdu zjffdu commented Aug 11, 2015

API is updated but its doc comment is not updated.

@rxin
Copy link
Contributor

rxin commented Aug 11, 2015

LGTM.

@rxin
Copy link
Contributor

rxin commented Aug 11, 2015

Jenkins, test this please.

@SparkQA
Copy link

SparkQA commented Aug 11, 2015

Test build #1443 has finished for PR 8097 at commit 3579729.

  • This patch fails PySpark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • case class SortMergeOuterJoin(

@srowen
Copy link
Member

srowen commented Aug 11, 2015

That error can't be related. All of the non-python tests passed and this doesn't affect Python. I think it's Ok to merge

@rxin
Copy link
Contributor

rxin commented Aug 11, 2015

yup - merging this. Thanks!

@asfgit asfgit closed this in bce7279 Aug 11, 2015
asfgit pushed a commit that referenced this pull request Aug 11, 2015
API is updated but its doc comment is not updated.

Author: Jeff Zhang <zjffdu@apache.org>

Closes #8097 from zjffdu/dev.

(cherry picked from commit bce7279)
Signed-off-by: Reynold Xin <rxin@databricks.com>
CodingCat pushed a commit to CodingCat/spark that referenced this pull request Aug 17, 2015
API is updated but its doc comment is not updated.

Author: Jeff Zhang <zjffdu@apache.org>

Closes apache#8097 from zjffdu/dev.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants