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-21551][Python] Increase timeout for PythonRDD.serveIterator #18752

Closed
wants to merge 1 commit into from

Conversation

peay
Copy link
Contributor

@peay peay commented Jul 27, 2017

What changes were proposed in this pull request?

This modification increases the timeout for serveIterator (which is not dynamically configurable). This fixes timeout issues in pyspark when using collect and similar functions, in cases where Python may take more than a couple seconds to connect.

See https://issues.apache.org/jira/browse/SPARK-21551

How was this patch tested?

Ran the tests.

cc @rxin

@rxin
Copy link
Contributor

rxin commented Jul 27, 2017

cc @JoshRosen

@peay
Copy link
Contributor Author

peay commented Aug 8, 2017

Any chance we can merge this?

@JoshRosen
Copy link
Contributor

This seems fine to me, especially since it's plausible that you might have a few-second GC pause in some situations. Let me go ahead and have Jenkins test this, then I'll merge it if tests pass (which I assume they will).

@JoshRosen
Copy link
Contributor

Jenkins, this is ok to test.

@SparkQA
Copy link

SparkQA commented Aug 9, 2017

Test build #80419 has finished for PR 18752 at commit 9d3c664.

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

@peay
Copy link
Contributor Author

peay commented Aug 9, 2017

Great, thanks @JoshRosen!

@rxin
Copy link
Contributor

rxin commented Aug 9, 2017

Merging in master.

@asfgit asfgit closed this in c06f3f5 Aug 9, 2017
asfgit pushed a commit that referenced this pull request Oct 19, 2017
Backport of #18752 (https://issues.apache.org/jira/browse/SPARK-21551)

(cherry picked from commit 9d3c664)

Author: peay <peay@protonmail.com>

Closes #19512 from FRosner/branch-2.2.
asfgit pushed a commit that referenced this pull request Oct 20, 2017
Backport of #18752 (https://issues.apache.org/jira/browse/SPARK-21551)

(cherry picked from commit 9d3c664)

Author: peay <peay@protonmail.com>

Closes #19513 from FRosner/branch-2.1.
asfgit pushed a commit that referenced this pull request Oct 21, 2017
Backport of #18752 (https://issues.apache.org/jira/browse/SPARK-21551)

(cherry picked from commit 9d3c664)

Author: peay <peay@protonmail.com>

Closes #19514 from FRosner/branch-2.0.
MatthewRBruce pushed a commit to Shopify/spark that referenced this pull request Jul 31, 2018
Backport of apache#18752 (https://issues.apache.org/jira/browse/SPARK-21551)

(cherry picked from commit 9d3c664)

Author: peay <peay@protonmail.com>

Closes apache#19512 from FRosner/branch-2.2.
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