From 20967852ef41df72ca7690e5e9a8dbf4807dc33e Mon Sep 17 00:00:00 2001
From: Nate Bauernfeind <natebauernfeind@deephaven.io>
Date: Wed, 15 Nov 2023 09:38:43 -0700
Subject: [PATCH] ExportObject PUBLISHING State Change Bug (#4835)

---
 .../java/io/deephaven/server/session/SessionState.java    | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/server/src/main/java/io/deephaven/server/session/SessionState.java b/server/src/main/java/io/deephaven/server/session/SessionState.java
index 2ba3045be61..56b98bbc751 100644
--- a/server/src/main/java/io/deephaven/server/session/SessionState.java
+++ b/server/src/main/java/io/deephaven/server/session/SessionState.java
@@ -682,7 +682,11 @@ private synchronized void setWork(
             this.errorHandler = errorHandler;
             this.successHandler = successHandler;
 
-            setState(ExportNotification.State.PENDING);
+            if (state != ExportNotification.State.PUBLISHING) {
+                setState(ExportNotification.State.PENDING);
+            } else if (dependentCount > 0) {
+                throw new IllegalStateException("published exports cannot have dependencies");
+            }
             if (dependentCount <= 0) {
                 dependentCount = 0;
                 scheduleExport();
@@ -920,7 +924,7 @@ private void onResolveOne(@Nullable final ExportObject<?> parent) {
          */
         private void scheduleExport() {
             synchronized (this) {
-                if (state != ExportNotification.State.PENDING) {
+                if (state != ExportNotification.State.PENDING && state != ExportNotification.State.PUBLISHING) {
                     return;
                 }
                 setState(ExportNotification.State.QUEUED);