Skip to content

Commit

Permalink
#2016: merge fix from 2.7.0 to 2.6.x (#2680)
Browse files Browse the repository at this point in the history
  • Loading branch information
beiwei30 authored Oct 24, 2018
1 parent a531a5c commit f58997f
Showing 1 changed file with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -124,11 +124,12 @@ public String toString() {

@Override
public Result invoke(Invocation inv) throws RpcException {
// if invoker is destroyed due to address refresh from registry, let's allow the current invoke to proceed
if (destroyed.get()) {
throw new RpcException("Rpc invoker for service " + this + " on consumer " + NetUtils.getLocalHost()
+ " use dubbo version " + Version.getVersion()
+ " is DESTROYED, can not be invoked any more!");
logger.warn("Invoker for service " + this + " on consumer " + NetUtils.getLocalHost() + " is destroyed, "

This comment has been minimized.

Copy link
@bert82503

bert82503 Jan 20, 2019

We encounter the same exception when dubbo shutdown. I does not understand why ignore the destroyed Invoker for Exporter unexported (AbstractExporter.unexport()). @beiwei30

+ ", dubbo version is " + Version.getVersion() + ", this invoker should not be used any longer");
}

RpcInvocation invocation = (RpcInvocation) inv;
invocation.setInvoker(this);
if (attachment != null && attachment.size() > 0) {
Expand Down

0 comments on commit f58997f

Please sign in to comment.