nizhikov commented on a change in pull request #9490:
URL: https://github.com/apache/ignite/pull/9490#discussion_r755808308
##########
File path:
modules/core/src/main/java/org/apache/ignite/internal/processors/platform/utils/PlatformUtils.java
##########
@@ -1063,12 +1066,17 @@ public static void initializeJavaObject(Object obj,
String clsName, @Nullable Ma
throw new IgniteException("Java object/factory class field
is not found [" +
"className=" + clsName + ", fieldName=" + fieldName +
']');
+ Object val = prop.getValue();
+
try {
- field.set(obj, prop.getValue());
+ field.set(
+ obj,
+ BinaryUtils.isObjectArray(field.getType()) ?
BinaryUtils.rawArrayFromBinary(val) : val
Review comment:
Right now we have single test for this feature in .Net -
[`ContinuousQueryJavaFilterTest#TestFilter`](https://github.com/apache/ignite/blob/master/modules/platforms/dotnet/Apache.Ignite.Core.Tests/Cache/Query/Continuous/ContinuousQueryJavaFilterTest.cs#L105)
and it seems feature not documented.
The changes just make test pass.
Do you think something should be additionally changed?
--
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]
