tkalkirill commented on code in PR #4602:
URL: https://github.com/apache/ignite-3/pull/4602#discussion_r1808554774
##########
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 like it better when we have a more strict structure, if there is an
opening bracket there should be a closing one, also with tags. For me, an
unclosed tag looks ugly. I can roll back this change if you insist, but in my
new code I will leave the closing tag.
--
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
