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-7836] [SPARK-7822] Python API of window functions #6364

Closed
wants to merge 6 commits into from

Conversation

davies
Copy link
Contributor

@davies davies commented May 22, 2015

This PR also fix ntile() to accept only integer.

Closes #6370

cc @rxin @yhuai

@since(1.4)
def ntile(n):
"""
NTILE for specified column or expression.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add "Window function:" ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will submit a patch to udpate scaladoc for all scala methods. the docs are not there yet.

@SparkQA
Copy link

SparkQA commented May 23, 2015

Test build #33367 has finished for PR 6364 at commit 778e2c0.

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

@rxin
Copy link
Contributor

rxin commented May 23, 2015

Looks like it failed in Python 3?

@SparkQA
Copy link

SparkQA commented May 23, 2015

Test build #33369 has finished for PR 6364 at commit 2649358.

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

@@ -130,12 +141,25 @@ def _(col1, col2):
'pow': 'Returns the value of the first argument raised to the power of the second argument.'
}

_window_functions = {
'lag': """returns the lag value of current row of the expression,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we update the documentation here?

Davies Liu and others added 2 commits May 22, 2015 18:31
@davies
Copy link
Contributor Author

davies commented May 23, 2015

@rxin Had updated

@SparkQA
Copy link

SparkQA commented May 23, 2015

Test build #33385 has finished for PR 6364 at commit 62bbb6c.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • class Window(object):
    • class WindowSpec(object):

@SparkQA
Copy link

SparkQA commented May 23, 2015

Test build #33386 has finished for PR 6364 at commit 66092b4.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • "public class " + className + extendsText + " implements java.io.Serializable
    • class Window(object):
    • class WindowSpec(object):

@SparkQA
Copy link

SparkQA commented May 23, 2015

Test build #853 has finished for PR 6364 at commit 62bbb6c.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@pwendell
Copy link
Contributor

I think this still needs sign off from @rxin

rxin added a commit to rxin/spark that referenced this pull request May 23, 2015
[SPARK-7836] [SPARK-7822] Python API of window functions
@davies
Copy link
Contributor Author

davies commented May 23, 2015

Yes, @rxin is working on it.

@rxin
Copy link
Contributor

rxin commented May 23, 2015

I submitted a pull request with more documentation changes: #6374

@davies davies closed this May 23, 2015
asfgit pushed a commit that referenced this pull request May 23, 2015
…elated updates

1. ntile should take an integer as parameter.
2. Added Python API (based on #6364)
3. Update documentation of various DataFrame Python functions.

Author: Davies Liu <davies@databricks.com>
Author: Reynold Xin <rxin@databricks.com>

Closes #6374 from rxin/window-final and squashes the following commits:

69004c7 [Reynold Xin] Style fix.
288cea9 [Reynold Xin] Update documentaiton.
7cb8985 [Reynold Xin] Merge pull request #6364 from davies/window
66092b4 [Davies Liu] update docs
ed73cb4 [Reynold Xin] [SPARK-7322][SQL] Improve DataFrame window function documentation.
ef55132 [Davies Liu] Merge branch 'master' of github.com:apache/spark into window4
8936ade [Davies Liu] fix maxint in python 3
2649358 [Davies Liu] update docs
778e2c0 [Davies Liu] SPARK-7836 and SPARK-7822: Python API of window functions

(cherry picked from commit efe3bfd)
Signed-off-by: Yin Huai <yhuai@databricks.com>
asfgit pushed a commit that referenced this pull request May 23, 2015
…elated updates

1. ntile should take an integer as parameter.
2. Added Python API (based on #6364)
3. Update documentation of various DataFrame Python functions.

Author: Davies Liu <davies@databricks.com>
Author: Reynold Xin <rxin@databricks.com>

Closes #6374 from rxin/window-final and squashes the following commits:

69004c7 [Reynold Xin] Style fix.
288cea9 [Reynold Xin] Update documentaiton.
7cb8985 [Reynold Xin] Merge pull request #6364 from davies/window
66092b4 [Davies Liu] update docs
ed73cb4 [Reynold Xin] [SPARK-7322][SQL] Improve DataFrame window function documentation.
ef55132 [Davies Liu] Merge branch 'master' of github.com:apache/spark into window4
8936ade [Davies Liu] fix maxint in python 3
2649358 [Davies Liu] update docs
778e2c0 [Davies Liu] SPARK-7836 and SPARK-7822: Python API of window functions
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request May 28, 2015
…elated updates

1. ntile should take an integer as parameter.
2. Added Python API (based on apache#6364)
3. Update documentation of various DataFrame Python functions.

Author: Davies Liu <davies@databricks.com>
Author: Reynold Xin <rxin@databricks.com>

Closes apache#6374 from rxin/window-final and squashes the following commits:

69004c7 [Reynold Xin] Style fix.
288cea9 [Reynold Xin] Update documentaiton.
7cb8985 [Reynold Xin] Merge pull request apache#6364 from davies/window
66092b4 [Davies Liu] update docs
ed73cb4 [Reynold Xin] [SPARK-7322][SQL] Improve DataFrame window function documentation.
ef55132 [Davies Liu] Merge branch 'master' of github.com:apache/spark into window4
8936ade [Davies Liu] fix maxint in python 3
2649358 [Davies Liu] update docs
778e2c0 [Davies Liu] SPARK-7836 and SPARK-7822: Python API of window functions
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request Jun 12, 2015
…elated updates

1. ntile should take an integer as parameter.
2. Added Python API (based on apache#6364)
3. Update documentation of various DataFrame Python functions.

Author: Davies Liu <davies@databricks.com>
Author: Reynold Xin <rxin@databricks.com>

Closes apache#6374 from rxin/window-final and squashes the following commits:

69004c7 [Reynold Xin] Style fix.
288cea9 [Reynold Xin] Update documentaiton.
7cb8985 [Reynold Xin] Merge pull request apache#6364 from davies/window
66092b4 [Davies Liu] update docs
ed73cb4 [Reynold Xin] [SPARK-7322][SQL] Improve DataFrame window function documentation.
ef55132 [Davies Liu] Merge branch 'master' of github.com:apache/spark into window4
8936ade [Davies Liu] fix maxint in python 3
2649358 [Davies Liu] update docs
778e2c0 [Davies Liu] SPARK-7836 and SPARK-7822: Python API of window functions
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
…elated updates

1. ntile should take an integer as parameter.
2. Added Python API (based on apache#6364)
3. Update documentation of various DataFrame Python functions.

Author: Davies Liu <davies@databricks.com>
Author: Reynold Xin <rxin@databricks.com>

Closes apache#6374 from rxin/window-final and squashes the following commits:

69004c7 [Reynold Xin] Style fix.
288cea9 [Reynold Xin] Update documentaiton.
7cb8985 [Reynold Xin] Merge pull request apache#6364 from davies/window
66092b4 [Davies Liu] update docs
ed73cb4 [Reynold Xin] [SPARK-7322][SQL] Improve DataFrame window function documentation.
ef55132 [Davies Liu] Merge branch 'master' of github.com:apache/spark into window4
8936ade [Davies Liu] fix maxint in python 3
2649358 [Davies Liu] update docs
778e2c0 [Davies Liu] SPARK-7836 and SPARK-7822: Python API of window functions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants