ascherbakoff commented on code in PR #4821:
URL: https://github.com/apache/ignite-3/pull/4821#discussion_r1891424333


##########
modules/core/src/main/java/org/apache/ignite/internal/hlc/HybridClock.java:
##########
@@ -44,7 +51,16 @@ public interface HybridClock {
     HybridTimestamp now();
 
     /**
-     * Gets a current timestamp. It is a fast way to get timestamp because it 
doesn't have to tick the logical part of the clock.
+     * Creates a timestamp for new event. A timestamp is guarantied to be 
unique and monotonically grown and follow the causal.
+     *
+     * @param causal The causal timestamp.
+     *
+     * @return The hybrid timestamp.
+     */
+    HybridTimestamp now(HybridTimestamp causal);

Review Comment:
   This method was removed, as it (unintentionally) duplicates "update"



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