ptupitsyn commented on a change in pull request #8867:
URL: https://github.com/apache/ignite/pull/8867#discussion_r596642483
##########
File path:
modules/core/src/main/java/org/apache/ignite/internal/binary/GridBinaryMarshaller.java
##########
@@ -150,9 +150,16 @@
/** Time array. */
public static final byte TIME_ARR = 37;
- /** Binary enum */
+ /** Binary enum. */
public static final byte BINARY_ENUM = 38;
+ /**
+ * Binary wrapper for {@code Object[]}.
+ * This wrapper used to store array component type id during serde process.
+ * @see BinaryUtils#doReadObjectArrayWrapper(BinaryInputStream,
BinaryContext, ClassLoader, BinaryReaderHandlesHolder, boolean, boolean)
+ */
+ public static final byte OBJ_ARR_WRAPPER = 39;
Review comment:
Agree with Alex, it would be great to avoid a new system type.
Otherwise we'll have to add the support for this type to .NET and C++, right?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
