Skip to content

[SPARK-20422][Spark Core] Worker registration retries should be configurable - #17738

Closed
unsleepy22 wants to merge 1 commit into
apache:masterfrom
unsleepy22:SPARK-20422
Closed

[SPARK-20422][Spark Core] Worker registration retries should be configurable#17738
unsleepy22 wants to merge 1 commit into
apache:masterfrom
unsleepy22:SPARK-20422

Conversation

@unsleepy22

Copy link
Copy Markdown

What changes were proposed in this pull request?

make prolonged registration retries configurable

How was this patch tested?

unit tests, integration tests

@AmplabJenkins

Copy link
Copy Markdown

Can one of the admins verify this patch?

@unsleepy22

Copy link
Copy Markdown
Author

Could someone take a look?

@navneetrastoginavneetrastogi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This looks ok to me

@navneetrastogi

Copy link
Copy Markdown

This is a minor change which will help in configuring registration retries. It can be merged.

@jiangxb1987

Copy link
Copy Markdown
Contributor

Could you expand more on the senerio you are changing this? Thank you!

@jiangxb1987jiangxb1987 mentioned this pull request Jun 25, 2017
@gatorsmile

Copy link
Copy Markdown
Member

We are closing it due to inactivity. please do reopen if you want to push it forward. Thanks!

zifeif2 pushed a commit to zifeif2/spark that referenced this pull request Nov 22, 2025
## What changes were proposed in this pull request?
This PR proposes to close stale PRs, mostly the same instances with apache#18017
I believe the author in apache#14807 removed his account.
Closesapache#7075Closesapache#8927Closesapache#9202Closesapache#9366Closesapache#10861Closesapache#11420Closesapache#12356Closesapache#13028Closesapache#13506Closesapache#14191Closesapache#14198Closesapache#14330Closesapache#14807Closesapache#15839Closesapache#16225Closesapache#16685Closesapache#16692Closesapache#16995Closesapache#17181Closesapache#17211Closesapache#17235Closesapache#17237Closesapache#17248Closesapache#17341Closesapache#17708Closesapache#17716Closesapache#17721Closesapache#17937
Added:
Closesapache#14739Closesapache#17139Closesapache#17445Closesapache#18042Closesapache#18359
Added:
Closesapache#16450Closesapache#16525Closesapache#17738
Added:
Closesapache#16458Closesapache#16508Closesapache#17714
Added:
Closesapache#17830Closesapache#14742
## How was this patch tested?
N/A
Author: hyukjinkwon <gurwls223@gmail.com>
Closesapache#18417 from HyukjinKwon/close-stale-pr.
Sign up for freeto 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.

5 participants

@unsleepy22@AmplabJenkins@navneetrastogi@jiangxb1987@gatorsmile