[GitHub] [kafka] C0urante commented on a diff in pull request #12984: KAFKA-14455: Kafka Connect create and update REST APIs should surface failures while writing to the config topic

[via GitHub]

C0urante commented on code in PR #12984:
URL: https://github.com/apache/kafka/pull/12984#discussion_r1093416934


##########
clients/src/main/java/org/apache/kafka/common/utils/Timer.java:
##########
@@ -161,7 +161,7 @@ public long remainingMs() {
     /**
      * Get the current time in milliseconds. This will return the same cached 
value until the timer
      * has been updated using one of the {@link #update()} methods or {@link 
#sleep(long)} is used.
-     *
+     * <p>

Review Comment:
   In the future, would you mind saving nonessential improvements like this for 
a dedicated PR? They tend to add noise to the diff and makes things harder to 
review.



-- 
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