-
Notifications
You must be signed in to change notification settings - Fork 28.7k
[SPARK-4172] [PySpark] Progress API in Python #3027
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
Conversation
Test build #22573 has started for PR 3027 at commit
|
Test build #22573 has finished for PR 3027 at commit
|
Test PASSed. |
I think maybe you meant to link to a different JIRA? |
@JoshRosen I had created a new JIRA for it. |
Test build #522 has started for PR 3027 at commit
|
Test build #522 has finished for PR 3027 at commit
|
@JoshRosen How does this look to you? |
Test build #23628 has started for PR 3027 at commit
|
Test build #23628 has finished for PR 3027 at commit
|
Test FAILed. |
Test build #534 has started for PR 3027 at commit
|
Test build #534 has finished for PR 3027 at commit
|
Conflicts: python/pyspark/__init__.py python/pyspark/context.py
Test build #26342 has started for PR 3027 at commit
|
Test build #26342 has finished for PR 3027 at commit
|
Test PASSed. |
Test build #27614 has started for PR 3027 at commit
|
Test build #27614 has finished for PR 3027 at commit
|
Test FAILed. |
Looks like this failed due to Scala style issues due to a few long lines. |
@JoshRosen I fixed it locally, but forgot to push out. |
Test build #612 has started for PR 3027 at commit
|
if (serializedData != null && serializedData.limit() > 0) { | ||
reason = serializer.get().deserialize[TaskEndReason]( | ||
serializedData, Utils.getSparkClassLoader) | ||
try { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not related change, it mute the exception when you cancel a job.
Test build #27637 has started for PR 3027 at commit
|
Test build #612 has finished for PR 3027 at commit
|
Test build #27637 has finished for PR 3027 at commit
|
Test PASSed. |
LGTM, so I'm going to merge this into |
This patch bring the pull based progress API into Python, also a example in Python. Author: Davies Liu <[email protected]> Closes #3027 from davies/progress_api and squashes the following commits: b1ba984 [Davies Liu] fix style d3b9253 [Davies Liu] add tests, mute the exception after stop 4297327 [Davies Liu] Merge branch 'master' of github.com:apache/spark into progress_api 969fa9d [Davies Liu] Merge branch 'master' of github.com:apache/spark into progress_api 25590c9 [Davies Liu] update with Java API 360de2d [Davies Liu] Merge branch 'master' of github.com:apache/spark into progress_api c0f1021 [Davies Liu] Merge branch 'master' of github.com:apache/spark into progress_api 023afb3 [Davies Liu] add Python API and example for progress API (cherry picked from commit 445a755) Signed-off-by: Josh Rosen <[email protected]>
This patch bring the pull based progress API into Python, also a example in Python.