Browse Source

bug about dolphinscheduler-remote SendAsync (#6972)

* bug about SendAsync
3.0.0/version-upgrade
lyq 3 years ago committed by GitHub
parent
commit
c78fd24776
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 4
      dolphinscheduler-remote/src/main/java/org/apache/dolphinscheduler/remote/future/ResponseFuture.java
  2. 1
      dolphinscheduler-remote/src/main/java/org/apache/dolphinscheduler/remote/handler/NettyClientHandler.java

4
dolphinscheduler-remote/src/main/java/org/apache/dolphinscheduler/remote/future/ResponseFuture.java

@ -105,6 +105,10 @@ public class ResponseFuture {
return FUTURE_TABLE.get(opaque); return FUTURE_TABLE.get(opaque);
} }
public void removeFuture() {
FUTURE_TABLE.remove(opaque);
}
/** /**
* whether timeout * whether timeout
* *

1
dolphinscheduler-remote/src/main/java/org/apache/dolphinscheduler/remote/handler/NettyClientHandler.java

@ -136,6 +136,7 @@ public class NettyClientHandler extends ChannelInboundHandlerAdapter {
future.setResponseCommand(command); future.setResponseCommand(command);
future.release(); future.release();
if (future.getInvokeCallback() != null) { if (future.getInvokeCallback() != null) {
future.removeFuture();
this.callbackExecutor.submit(future::executeInvokeCallback); this.callbackExecutor.submit(future::executeInvokeCallback);
} else { } else {
future.putResponse(command); future.putResponse(command);

Loading…
Cancel
Save