fix a npe bug after refactor recycler of BookieClient

fanjianye · fanjianye · commit d77caa053aca · 2025-06-04T16:33:52.000+08:00
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) {`