diff options
author | Björn Gustavsson <[email protected]> | 2016-03-11 13:13:59 +0100 |
---|---|---|
committer | Björn Gustavsson <[email protected]> | 2016-03-11 14:41:55 +0100 |
commit | cccf762d93640b707f8e1515604cebcfbe379c04 (patch) | |
tree | 2c9ccb69330dc490ccb9970bdf79861a6512d411 /otp_build | |
parent | e0bb95ed1e7e1314697b51d10ae59cd09c557941 (diff) | |
download | otp-cccf762d93640b707f8e1515604cebcfbe379c04.tar.gz otp-cccf762d93640b707f8e1515604cebcfbe379c04.tar.bz2 otp-cccf762d93640b707f8e1515604cebcfbe379c04.zip |
Don't throw exceptions from rpc:call() and rpc:block_call()
The documentation for rpc:call() says:
Evaluates apply(Module, Function, Args) on the node Node and
returns the corresponding value Res, or {badrpc, Reason} if
the call fails.
What is not said that rpc:call() can generate an exception if the
'rex' process on the other node is killed:
(kalle@host)1> Rex = rpc:call(arne@host, erlang, whereis, [rex]).
<6937.14.0>
(kalle@host)2> rpc:call(arne@host, erlang, exit, [Rex,kill]).
** exception exit: {killed,
{gen_server,call,
[{rex,arne@host},
{call,erlang,exit,[<6937.14.0>,kill],<0.33.0>},
infinity]}}
in function rpc:rpc_check/1 (rpc.erl, line 361)
On the other hand, if the other node shuts down for some other reason,
we'll get a {badrpc,nodedown} result:
(kalle@host)5> rpc:call(arne@host, erlang, halt, []).
{badrpc,nodedown}
There does not seem to be any reason to handle the two cases
differently. If the 'rex' process is terminated on the other node,
it will shut down shortly thereafter.
Therefore, change rpc:call() and rpc:block_call() to always return
{badrpc,Reason} is the call fails:
(kalle@host)1> Rex = rpc:call(arne@host, erlang, whereis, [rex]).
<6937.14.0>
(kalle@host)2> rpc:call(arne@host, erlang, exit, [Rex,kill]).
{badrpc,{'EXIT',{killed,{gen_server,call,
[{rex,arne@host},
{call,erlang,exit,[<7126.14.0>,kill],<0.33.0>},
infinity]}}}}
Diffstat (limited to 'otp_build')
0 files changed, 0 insertions, 0 deletions