fix a npe bug after refactor recycler of BookieClient (#4610)

TakaHiR07 · fanjianye · StevenLuMT · commit 98323185ffd8 · 2025-06-12T10:39:53.000+08:00
Co-authored-by: fanjianye <fanjianye@bigo.sg> (cherry picked from commit 23289b7)
diff --git a/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/CompletionValue.java b/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/CompletionValue.java
@@ -87,9 +87,9 @@ boolean maybeTimeout() {
     }
 
     void timeout() {
-        errorOut(BKException.Code.TimeoutException);
         timeoutOpLogger.registerSuccessfulEvent(latency(),
                 TimeUnit.NANOSECONDS);
+        errorOut(BKException.Code.TimeoutException);
     }
 
     protected void logResponse(BookkeeperProtocol.StatusCode status, Object... extraInfo) {

Original file line number	Diff line number	Diff line change
`@@ -87,9 +87,9 @@ boolean maybeTimeout() {`
`87`	`87`	`}`
`88`	`88`
`89`	`89`	`void timeout() {`
`90`		`- errorOut(BKException.Code.TimeoutException);`
`91`	`90`	`timeoutOpLogger.registerSuccessfulEvent(latency(),`
`92`	`91`	`TimeUnit.NANOSECONDS);`
	`92`	`+ errorOut(BKException.Code.TimeoutException);`
`93`	`93`	`}`
`94`	`94`
`95`	`95`	`protected void logResponse(BookkeeperProtocol.StatusCode status, Object... extraInfo) {`