Skip to content

[SPARK-19163][PYTHON][SQL] Delay _judf initialization to the __call__ #16536

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.

Sign up for GitHub

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

Closed
zero323 wants to merge 8 commits into from
Closed

[SPARK-19163][PYTHON][SQL] Delay _judf initialization to the __call__ #16536

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
22b967c
Delay _judf initialization to the __call__
zero323 Jan 7, 2017
7b252c6
Move name resolution logic from _create_judf to __init__
zero323 Jan 13, 2017
fb009e0
Use getOrCreate instead of _active_spark_context
zero323 Jan 26, 2017
489ef54
Add UDFInitializationTests
zero323 Jan 30, 2017
7333f29
Add comment describing multithreaded initalization
zero323 Jan 30, 2017
cb496f3
Use _active_spark_context in place of getOrCreate
zero323 Feb 1, 2017
9332da3
Add a comment explaining the purpose of _judf_placeholder
zero323 Feb 1, 2017
923b88d
Use spark.sparkContext instead of separate _judf
zero323 Feb 1, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Use _active_spark_context in place of getOrCreate
  • Loading branch information
@zero323
zero323 committed Feb 1, 2017
commit cb496f3e5384f822b7e254a808af3ddae2dd6ff6
6 changes: 3 additions & 3 deletions python/pyspark/sql/functions.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1854,9 +1854,9 @@ def _create_judf(self):
return judf

def __call__(self, *cols):
sc = SparkContext.getOrCreate()
jc = self._judf.apply(_to_seq(sc, cols, _to_java_column))
return Column(jc)
judf = self._judf
sc = SparkContext._active_spark_context
return Column(judf.apply(_to_seq(sc, cols, _to_java_column)))


@since(1.3)
Expand Down