ascherbakoff commented on code in PR #4929:
URL: https://github.com/apache/ignite-3/pull/4929#discussion_r1898420260
##########
modules/transactions/src/main/java/org/apache/ignite/internal/tx/HybridTimestampTracker.java:
##########
@@ -17,36 +17,102 @@
package org.apache.ignite.internal.tx;
-import static
org.apache.ignite.internal.hlc.HybridTimestamp.NULL_HYBRID_TIMESTAMP;
+import static
org.apache.ignite.internal.hlc.HybridTimestamp.hybridTimestampToLong;
+import static
org.apache.ignite.internal.hlc.HybridTimestamp.nullableHybridTimestamp;
import java.util.concurrent.atomic.AtomicLong;
+import java.util.function.Consumer;
import org.apache.ignite.internal.hlc.HybridTimestamp;
import org.jetbrains.annotations.Nullable;
/**
- * Hybrid timestamp tracker.
+ * Interface is used to provide a track timestamp into a transaction operation.
*/
-public class HybridTimestampTracker {
- /** Timestamp. */
- private final AtomicLong timestamp = new AtomicLong(NULL_HYBRID_TIMESTAMP);
+public abstract class HybridTimestampTracker {
Review Comment:
Why this is not an interface ?
--
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
