[GitHub] [ignite] nizhikov commented on pull request #9630: IGNITE-16057 Second read of array field from BinaryObject fail

[GitBox <gi...@apache.org>]

nizhikov commented on pull request #9630:
URL: https://github.com/apache/ignite/pull/9630#issuecomment-992286639


   @vladErmakov07 Can you, please, extends your fix with the following:
   
   1. Extends tests with the `Collection`, `Map` and multidimensional array.
   2. Add the following scenario:
   
   ```
   private class TestClass {
       private MyClass field;
   
       private MyClass[] array;
   }
   
   //----
   
   TestClass t = new TestClass();
   
   t.field = new MyClass();
   t.array = new MyClass[] { new MyClass(), t.field, new Myclass() }
   
   TestClass t2 = serde(t);
   
   assertSame(t2.field, t2.array[1]);
   ```
   
   I think the root cause of the issue is - we don't move stream pointer to the end of the object when we read it with the handle. 


-- 
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: notifications-unsubscribe@ignite.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org