From f9ee26a100684ffaec581e42e79b1ee300d9724d Mon Sep 17 00:00:00 2001 From: Nate Bauernfeind Date: Tue, 9 Jan 2024 11:05:59 -0700 Subject: [PATCH] Fix Server-Side ABORT That Should be CANCELLED (#5018) --- .../server/appmode/ApplicationServiceGrpcImpl.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/server/src/main/java/io/deephaven/server/appmode/ApplicationServiceGrpcImpl.java b/server/src/main/java/io/deephaven/server/appmode/ApplicationServiceGrpcImpl.java index d31b26070c4..c40fcf913ff 100644 --- a/server/src/main/java/io/deephaven/server/appmode/ApplicationServiceGrpcImpl.java +++ b/server/src/main/java/io/deephaven/server/appmode/ApplicationServiceGrpcImpl.java @@ -145,7 +145,7 @@ public synchronized void listFields( synchronized void remove(Subscription sub) { if (subscriptions.remove(sub)) { - sub.notifyObserverAborted(); + sub.notifyObserverCancelled(); } } @@ -250,8 +250,8 @@ private boolean send(FieldsChangeUpdate changes) { } // must be sync wrt parent - private void notifyObserverAborted() { - GrpcUtil.safelyError(observer, Code.ABORTED, "subscription cancelled"); + private void notifyObserverCancelled() { + GrpcUtil.safelyError(observer, Code.CANCELLED, "subscription cancelled"); } }