Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-9978] [PySpark] [SQL] fix Window.orderBy and doc of ntile() #8213

Closed
wants to merge 1 commit into from

Conversation

davies
Copy link
Contributor

@davies davies commented Aug 14, 2015

No description provided.

@davies
Copy link
Contributor Author

davies commented Aug 14, 2015

cc @rxin

@rxin
Copy link
Contributor

rxin commented Aug 14, 2015

LGTM

@SparkQA
Copy link

SparkQA commented Aug 14, 2015

Test build #1608 has finished for PR 8213 at commit 56cf18e.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • class StringIndexerModel (
    • implicit class StringToColumn(val sc: StringContext)

@rxin
Copy link
Contributor

rxin commented Aug 14, 2015

I've merged this.

asfgit pushed a commit that referenced this pull request Aug 14, 2015
Author: Davies Liu <davies@databricks.com>

Closes #8213 from davies/fix_window.

(cherry picked from commit 11ed2b1)
Signed-off-by: Reynold Xin <rxin@databricks.com>
asfgit pushed a commit that referenced this pull request Aug 14, 2015
Author: Davies Liu <davies@databricks.com>

Closes #8213 from davies/fix_window.

(cherry picked from commit 11ed2b1)
Signed-off-by: Reynold Xin <rxin@databricks.com>
@rxin
Copy link
Contributor

rxin commented Aug 14, 2015

in master/branch-1.5/branch-1.4.

@asfgit asfgit closed this in 11ed2b1 Aug 14, 2015
CodingCat pushed a commit to CodingCat/spark that referenced this pull request Aug 17, 2015
Author: Davies Liu <davies@databricks.com>

Closes apache#8213 from davies/fix_window.
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.

3 participants