From feb3a24f0620af65c9234da200fbaf81b1a6c0ec Mon Sep 17 00:00:00 2001
From: Greg Wilkins <gregw@webtide.com>
Date: Mon, 13 Sep 2021 11:22:20 +1000
Subject: [PATCH] Added puml diagram

More execution strategy ideas

Signed-off-by: Greg Wilkins <gregw@webtide.com>
---
 .../eclipse/jetty/quic/server/H3Server.puml   | 37 ++++++++++++++-----
 1 file changed, 27 insertions(+), 10 deletions(-)

diff --git a/jetty-quic/quic-server/src/main/java/org/eclipse/jetty/quic/server/H3Server.puml b/jetty-quic/quic-server/src/main/java/org/eclipse/jetty/quic/server/H3Server.puml
index 87a79d3a45e..05f2397d936 100644
--- a/jetty-quic/quic-server/src/main/java/org/eclipse/jetty/quic/server/H3Server.puml
+++ b/jetty-quic/quic-server/src/main/java/org/eclipse/jetty/quic/server/H3Server.puml
@@ -84,6 +84,13 @@ loop as packets received
               loop while task==null
                 alt if instruction available
                   QuicSession -> Decoder : takeInstruction
+                    note right
+                      I'm not sure how we ensure a taken instruction
+                      is always written?  Do we queue them or only take them
+                      when writable?  If we queue, then perhaps they are just
+                      queued during production rather than returned as produced
+                      tasks
+                    end note
                   return task
                 else if metadata available
                   QuicStream -> Decoder : takeMetaData
@@ -98,19 +105,29 @@ loop as packets received
                 end
               end
             else normal stream
-              QuicStream -> QuicSession : fill
-                QuicSession -> Quiche : drainClearText\nForStream
-                return bytes filled
-              return bytes filled
-              QuicStream -> QuicStream : parse
-                alt if header frame
-                  QuicStream -> Decoder : decode
-                  return
+              loop task==null
+                alt if instruction available
+                  QuicStream -> Decoder : takeInstruction
+                  return task
+                else if metadata available
                   QuicStream -> Decoder : takeMetaData
                   return task
-                else data frame
+                else
+                  QuicStream -> QuicSession : fill
+                    QuicSession -> Quiche : drainClearText\nForStream
+                    return bytes filled
+                  return bytes filled
+                  QuicStream -> QuicStream : parse
+                    alt if header frame
+                      QuicStream -> Decoder : decode
+                      return
+                    else data frame
+                      QuicStream -> HttpChannel : getCallback
+                      return task
+                    end
+                  return
                 end
-              return task
+              end
             end
           return task
         end