poorbarcode commented on code in PR #16165:
URL: https://github.com/apache/pulsar/pull/16165#discussion_r905846208


##########
pulsar-client/src/main/java/org/apache/pulsar/client/impl/ClientCnx.java:
##########
@@ -1283,17 +1283,13 @@ public boolean alreadyRelease(){
     }
 
     /**
-     * Changes the idle-state of the connection to #{@link 
IdleState#IDLE_MARKED}, This method only changes this
-     * connection from the #{@link IdleState#USING} state to the #{@link 
IdleState#IDLE_MARKED} state. if the
-     * idle-status is successfully changed, "idleMarkTime" is changed to 
current time.
-     * @return Whether change idle-stat to #{@link IdleState#IDLE_MARKED} 
success.
+     * Try to transform the state of the connection to #{@link 
IdleState#IDLE_MARKED}, state should only be transformed to #{@link 
IdleState#IDLE_MARKED} from state  #{@link IdleState#USING}. if the
+     * state is successfully transformed, "idleMarkTime" will be  assigned to 
current time.

Review Comment:
   Hi @gaozhangmin 
   src/main/java/org/apache/pulsar/client/impl/ClientCnx.java:[1286] (sizes) 
LineLength: Line is longer than 120 characters



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to