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