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-22769][CORE] When driver stopping, there is errors: Could not find CoarseGrainedScheduler and RpcEnv already stopped #19968
Conversation
You already had another PR open for the same issue, which was an exact duplicate of another PR, for another issue that is a duplicate. Please clean it up into one issue -- SPAKR-22769 |
Test build #84855 has finished for PR 19968 at commit
|
Test build #84856 has finished for PR 19968 at commit
|
Test build #84867 has finished for PR 19968 at commit
|
@@ -100,6 +102,7 @@ private[netty] class Dispatcher(nettyEnv: NettyRpcEnv, numUsableCores: Int) exte | |||
return | |||
} | |||
unregisterRpcEndpoint(rpcEndpointRef.name) | |||
endpointsIsStopped.putIfAbsent(rpcEndpointRef.name, true) |
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 will be a memory leak. Spark may create some temp RpcEndpoints. You can make postMessage
return Some(RpcEndpointNotFoundException)
and ignore it.
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.
ok thanks
Ping @KaiXinXiaoLei to update or close |
@srowen ok ,i will update, thanks |
ping @KaiXinXiaoLei are you still working on this? |
Now this problem, i don't work. Now i close it . |
What changes were proposed in this pull request?
When driver stopping, there is a error:
Because the endpoint of CoarseGrainedScheduler is stopped, this error is not problem, so i think this error should not be printed.
And there is another error:
I think the log level should be warning, not error.
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)
Please review http://spark.apache.org/contributing.html before opening a pull request.