AMashenkov commented on a change in pull request #739:
URL: https://github.com/apache/ignite-3/pull/739#discussion_r833030337



##########
File path: 
modules/storage-api/src/main/java/org/apache/ignite/internal/storage/PartitionStorage.java
##########
@@ -128,6 +143,66 @@
      */
     Cursor<DataRow> scan(Predicate<SearchRow> filter) throws StorageException;
 
+    /**
+     * Scans the partition and returns a cursor of values in at the given 
timestamp.
+     *
+     * @param keyFilter Key filter. Binary rows passed to the filter may or 
may not have a value, filter should only check keys.
+     * @param timestamp Timestamp
+     * @return Cursor.
+     */
+    default Cursor<BinaryRow> scan(Predicate<BinaryRow> keyFilter, @Nullable 
Timestamp timestamp) {
+        throw new UnsupportedOperationException("scan");
+    }
+
+    /**
+     * Exception class that describes the situation where two independant 
transactions attempting to write values for the same key.
+     */
+    class TxIdMismatchException extends RuntimeException {
+    }

Review comment:
       Let's move to a separate high-level class (or make is static at least).




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


Reply via email to