From 3312ef0aa23999782264f91fd550b4298be8e4b4 Mon Sep 17 00:00:00 2001
From: akarnokd <akarnokd@gmail.com>
Date: Thu, 6 Apr 2017 15:13:35 +0200
Subject: [PATCH] Refactor AbstractEmitterSubscriber

---
 .../annotation/ReactiveTypeHandler.java       | 115 ++++++++++--------
 1 file changed, 62 insertions(+), 53 deletions(-)

diff --git a/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/ReactiveTypeHandler.java b/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/ReactiveTypeHandler.java
index bd9dab7025..1104856622 100644
--- a/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/ReactiveTypeHandler.java
+++ b/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/ReactiveTypeHandler.java
@@ -19,16 +19,14 @@ import java.io.IOException;
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Optional;
-import java.util.Queue;
-import java.util.concurrent.ConcurrentLinkedQueue;
-import java.util.concurrent.atomic.AtomicBoolean;
+import java.util.concurrent.atomic.AtomicLong;
+import java.util.concurrent.atomic.AtomicReference;
 
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 import org.reactivestreams.Publisher;
 import org.reactivestreams.Subscriber;
 import org.reactivestreams.Subscription;
-
 import org.springframework.core.MethodParameter;
 import org.springframework.core.ReactiveAdapter;
 import org.springframework.core.ReactiveAdapterRegistry;
@@ -178,10 +176,7 @@ class ReactiveTypeHandler {
 	}
 
 
-	private static abstract class AbstractEmitterSubscriber implements Subscriber<Object> {
-
-		private static final Object COMPLETE_SIGNAL = new Object();
-
+	private static abstract class AbstractEmitterSubscriber implements Subscriber<Object>, Runnable {
 
 		private final ResponseBodyEmitter emitter;
 
@@ -189,12 +184,15 @@ class ReactiveTypeHandler {
 
 		private Subscription subscription;
 
-		private final Queue<Object> queue = new ConcurrentLinkedQueue<>();
+		private final AtomicReference<Object> queue = new AtomicReference<Object>();
 
-		private final AtomicBoolean executing = new AtomicBoolean(false);
+		private final AtomicLong executing = new AtomicLong();
 
 		private volatile boolean done;
-
+		
+		private volatile boolean terminated;
+		
+		private Throwable error;
 
 		protected AbstractEmitterSubscriber(ResponseBodyEmitter emitter, TaskExecutor executor) {
 			this.emitter = emitter;
@@ -231,69 +229,57 @@ class ReactiveTypeHandler {
 
 		@Override
 		public final void onNext(Object element) {
-			this.queue.offer(element);
+			this.queue.lazySet(element);
 			trySchedule();
 		}
 
 		@Override
 		public final void onError(Throwable ex) {
-			this.queue.offer(ex);
+			error = ex;
+			terminated = true;
 			trySchedule();
 		}
 
 		@Override
 		public final void onComplete() {
-			this.queue.offer(COMPLETE_SIGNAL);
+			terminated = true;
 			trySchedule();
 		}
 
 		private void trySchedule() {
-			if (this.executing.compareAndSet(false, true)) {
+			if (this.executing.getAndIncrement() == 0) {
+				schedule();
+			}
+		}
+		
+		private void schedule() {
+			try {
+				this.taskExecutor.execute(this);
+			}
+			catch (Throwable ex) {
 				try {
-					this.taskExecutor.execute(() -> {
-						try {
-							Object signal = this.queue.poll();
-							if (!this.done) {
-								handle(signal);
-							}
-						}
-						finally {
-							this.executing.set(false);
-							if(!this.queue.isEmpty())
-								trySchedule();
-						}
-					});
+					terminate();
 				}
-				catch (Throwable ex) {
-					try {
-						terminate();
-					}
-					finally {
-						this.executing.set(false);
-						this.queue.clear();
-					}
+				finally {
+					this.executing.decrementAndGet();
+					queue.lazySet(null);
 				}
 			}
 		}
-
-		private void handle(Object signal) {
-			if (signal instanceof Throwable) {
-				if (logger.isDebugEnabled()) {
-					logger.debug("Publisher error for " + this.emitter, (Throwable) signal);
-				}
-				this.done = true;
-				this.emitter.completeWithError((Throwable) signal);
+		
+		@Override
+		public void run() {
+			if (done) {
+				queue.lazySet(null);
+				return;
 			}
-			else if (signal == COMPLETE_SIGNAL) {
-				if (logger.isDebugEnabled()) {
-					logger.debug("Publishing completed for " + this.emitter);
-				}
-				this.done = true;
-				this.emitter.complete();
-			}
-			else {
+				
+			boolean d = terminated;
+			Object o = queue.get();
+			if (o != null) {
+				queue.lazySet(null);
 				try {
-					send(signal);
+					send(o);
 					this.subscription.request(1);
 				}
 				catch (final Throwable ex) {
@@ -301,8 +287,31 @@ class ReactiveTypeHandler {
 						logger.debug("Send error for " + this.emitter, ex);
 					}
 					terminate();
+					return;
 				}
 			}
+			
+			if (d) {
+				this.done = true;
+				Throwable ex = error;
+				error = null;
+				if (ex != null) {
+					if (logger.isDebugEnabled()) {
+						logger.debug("Publisher error for " + this.emitter, ex);
+					}
+					emitter.completeWithError(ex);
+				} else {
+					if (logger.isDebugEnabled()) {
+						logger.debug("Publishing completed for " + this.emitter);
+					}
+					this.emitter.complete();
+				}
+				return;
+			}
+			
+			if (executing.decrementAndGet() != 0) {
+				schedule();
+			}
 		}
 
 		protected abstract void send(Object element) throws IOException;