sashapolo commented on code in PR #4602:
URL: https://github.com/apache/ignite-3/pull/4602#discussion_r1808626583


##########
modules/metastorage/src/main/java/org/apache/ignite/internal/metastorage/server/WatchProcessor.java:
##########
@@ -335,7 +335,7 @@ private void invokeOnRevisionCallback(long revision, 
HybridTimestamp time) {
     /**
      * Advances safe time without notifying watches (as there is no new 
revision).
      *
-     * <p>This method is not thread-safe and must be performed under an 
exclusive lock in concurrent scenarios.
+     * <p>This method is not thread-safe and must be performed under an 
exclusive lock in concurrent scenarios.</p>

Review Comment:
   I think it's better to be consistent (i.e. not to use `</p>` tags 
everywhere), but I know that this discussion will not lead anywhere. So please 
at least keep the existing code intact.



-- 
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: notifications-unsubscr...@ignite.apache.org

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

Reply via email to