[#9595] Improve async state propagation in NonSampling state

emeroad · emeroad · commit 1a43c3c6d929 · 2023-01-26T17:27:23.000+09:00
diff --git a/profiler/src/main/java/com/navercorp/pinpoint/profiler/context/DefaultCloseListener.java b/profiler/src/main/java/com/navercorp/pinpoint/profiler/context/DefaultCloseListener.java
@@ -30,7 +30,8 @@ public void close(long endTime) {
 
 
     private void recordUriTemplate(long afterTime) {
-        if (uriStatStorage == null) {
+        final UriStatStorage copy = uriStatStorage;
+        if (copy == null) {
             return;
         }
 
@@ -39,7 +40,7 @@ private void recordUriTemplate(long afterTime) {
         long traceStartTime = traceRoot.getTraceStartTime();
 
         boolean status = getStatus(shared.getErrorCode());
-        uriStatStorage.store(uriTemplate, status, traceStartTime, afterTime);
+        copy.store(uriTemplate, status, traceStartTime, afterTime);
     }
 
     private boolean getStatus(int errorCode) {

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,8 @@ public void close(long endTime) {`
`30`	`30`
`31`	`31`
`32`	`32`	`private void recordUriTemplate(long afterTime) {`
`33`		`- if (uriStatStorage == null) {`
	`33`	`+ final UriStatStorage copy = uriStatStorage;`
	`34`	`+ if (copy == null) {`
`34`	`35`	`return;`
`35`	`36`	`}`
`36`	`37`
`@@ -39,7 +40,7 @@ private void recordUriTemplate(long afterTime) {`
`39`	`40`	`long traceStartTime = traceRoot.getTraceStartTime();`
`40`	`41`
`41`	`42`	`boolean status = getStatus(shared.getErrorCode());`
`42`		`- uriStatStorage.store(uriTemplate, status, traceStartTime, afterTime);`
	`43`	`+ copy.store(uriTemplate, status, traceStartTime, afterTime);`
`43`	`44`	`}`
`44`	`45`
`45`	`46`	`private boolean getStatus(int errorCode) {`