tkalkirill commented on code in PR #1003:
URL: https://github.com/apache/ignite-3/pull/1003#discussion_r945529418
##########
modules/storage-api/src/main/java/org/apache/ignite/internal/storage/RowId.java:
##########
@@ -17,14 +17,96 @@
package org.apache.ignite.internal.storage;
+import java.util.UUID;
+import org.apache.ignite.internal.tx.Timestamp;
+
/**
- * Interface that represents row id in primary index of the table.
+ * Class that represents row id in primary index of the table. Contains a
timestamp-based UUID and a partition id.
*
* @see MvPartitionStorage
*/
-public interface RowId {
+public final class RowId {
+ /** Partition id. */
Review Comment:
Please describe why **short** and not **int**.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]