Skip to content

Commit

Permalink
fix dubbo proxyless xds connect, fix Connection closed after GOAWAY, …
Browse files Browse the repository at this point in the history
…add xds observer retry create (apache#10544)
  • Loading branch information
wucheng1997 authored Oct 10, 2022
1 parent f3e999e commit d84d098
Showing 1 changed file with 12 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -154,6 +154,11 @@ public long observeResource(Set<String> resourceNames, Consumer<T> consumer) {
// observer reused
StreamObserver<DiscoveryRequest> observer = requestObserverMap.get(request);

if (observer == null) {
observer = xdsChannel.createDeltaDiscoveryRequest(new ResponseObserver(request));
requestObserverMap.put(request, observer);
}

// send request to control panel
observer.onNext(buildDiscoveryRequest(names));

Expand Down Expand Up @@ -231,11 +236,17 @@ public void onNext(DiscoveryResponse value) {
@Override
public void onError(Throwable t) {
logger.error("xDS Client received error message! detail:", t);
clear();
}

@Override
public void onCompleted() {
// ignore
logger.info("xDS Client completed, requestId: " + requestId);
clear();
}

private void clear() {
requestObserverMap.remove(requestId);
}
}

Expand Down

0 comments on commit d84d098

Please sign in to comment.