🐛 set prefetchedVariables to null instead of empty map.

xfeep · xfeep · commit 6082b664f4ba · 2019-11-26T16:48:31.000+08:00
diff --git a/src/java/nginx/clojure/clj/LazyRequestMap.java b/src/java/nginx/clojure/clj/LazyRequestMap.java
@@ -578,8 +578,8 @@ public void tagReleased() {
 			}
 			
 			if (this.prefetchedVariables != null) {
-				this.prefetchedVariables.clear();
-				this.updatedVariables.clear();
+				this.prefetchedVariables = null;
+				this.updatedVariables = null;
 			}
 						
 			((NginxClojureHandler)handler).returnToRequestPool(this);
diff --git a/src/java/nginx/clojure/java/NginxJavaRequest.java b/src/java/nginx/clojure/java/NginxJavaRequest.java
@@ -536,8 +536,8 @@ public void tagReleased() {
 		}
 		
 		if (prefetchedVariables != null) {
-			prefetchedVariables.clear();
-			updatedVariables.clear();
+			prefetchedVariables = null;
+			updatedVariables = null;
 		}
 		
 		((NginxJavaHandler)handler).returnToRequestPool(this);
diff --git a/test/nginx-working-dir/conf/nginx-plain.conf b/test/nginx-working-dir/conf/nginx-plain.conf
@@ -195,7 +195,7 @@ http {
 
     
     server {
-        listen       8080 reuseport;
+        listen       8080;
         server_name  localhost;
        
        #uncomment this two lines for performance test 

Original file line number	Diff line number	Diff line change
`@@ -578,8 +578,8 @@ public void tagReleased() {`
`578`	`578`	`}`
`579`	`579`
`580`	`580`	`if (this.prefetchedVariables != null) {`
`581`		`- this.prefetchedVariables.clear();`
`582`		`- this.updatedVariables.clear();`
	`581`	`+ this.prefetchedVariables = null;`
	`582`	`+ this.updatedVariables = null;`
`583`	`583`	`}`
`584`	`584`
`585`	`585`	`((NginxClojureHandler)handler).returnToRequestPool(this);`
Original file line number	Diff line number	Diff line change
`@@ -536,8 +536,8 @@ public void tagReleased() {`
`536`	`536`	`}`
`537`	`537`
`538`	`538`	`if (prefetchedVariables != null) {`
`539`		`- prefetchedVariables.clear();`
`540`		`- updatedVariables.clear();`
	`539`	`+ prefetchedVariables = null;`
	`540`	`+ updatedVariables = null;`
`541`	`541`	`}`
`542`	`542`
`543`	`543`	`((NginxJavaHandler)handler).returnToRequestPool(this);`