rpuch commented on code in PR #1464:
URL: https://github.com/apache/ignite-3/pull/1464#discussion_r1056086777
##########
modules/storage-api/src/testFixtures/java/org/apache/ignite/internal/storage/BaseMvPartitionStorageTest.java:
##########
@@ -47,9 +47,9 @@ public abstract class BaseMvPartitionStorageTest extends
BaseMvStoragesTest {
protected static final TestKey KEY = new TestKey(10, "foo");
- protected static final TestValue VALUE = new TestValue(20, "bar");
+ protected static final BinaryRow BINARY_ROW = binaryRow(KEY, new
TestValue(20, "bar"));
- protected static final BinaryRow BINARY_ROW = binaryRow(KEY, VALUE);
+ protected static final BinaryRow BINARY_ROW2 = binaryRow(KEY, new
TestValue(20, "bar"));
Review Comment:
It looks like the values of `BINARY_ROW` and `BINARY_ROW2` are identical.
They should probably be different.
--
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]
