zstan commented on code in PR #13391:
URL: https://github.com/apache/ignite/pull/13391#discussion_r3795548708
##########
modules/core/src/test/java/org/apache/ignite/internal/binary/BinaryMarshallerSelfTest.java:
##########
@@ -126,6 +134,46 @@ public void testNull() throws Exception {
assertNull(marshalUnmarshal(null));
}
+ /**
+ * @throws Exception If failed.
+ */
+ @Test
+ public void testSerializableRecord() throws Exception {
Review Comment:
apache ignite master currently support only 17 java - thus i suppose you can
simplify this test and rewrite it like other presented here ? Also you need
this PR agains master branch, not against ignite-2.17
--
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]
