Fix Server-Side ABORT That Should be CANCELLED (#5018)

deephaven · Jan 9, 2024 · f9ee26a · f9ee26a
1 parent 3229d71
commit f9ee26a
Showing 1 changed file with 3 additions and 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
@@ -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");
         }
     }