vamossagar12 commented on code in PR #12826:
URL: https://github.com/apache/kafka/pull/12826#discussion_r1016131415


##########
clients/src/main/java/org/apache/kafka/common/utils/Timer.java:
##########
@@ -172,6 +172,17 @@ public long currentTimeMs() {
         return currentTimeMs;
     }
 
+    /**
+     * Get the deadline time in milliseconds when this timer would expire. The 
deadlineMs time
+     * is generally updated through {@link #reset(long)} method or {@link 
#updateAndReset(long)}
+     * method call.
+     *
+     * @return The deadline time in milliseconds at which the timer would 
expire.
+     */
+    public long deadlineMs() {
+        return deadlineMs;
+    }
+

Review Comment:
   hmm maybe it wasn't needed before? I see no difference in exposing this v/s 
exposing `currentTimeMs` for example. Let me know what you think about it?



-- 
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: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to