anton-vinogradov commented on PR #13236: URL: https://github.com/apache/ignite/pull/13236#issuecomment-4812342907
I put the `QueryIndex` counterpart of the new serde test up as a PR against your branch — shishkovilja/ignite#27. It mirrors `QueryEntityMessageSerializationTest`: a full `DirectMessageWriter`/`DirectMessageReader` round-trip of `QueryIndexMessage` through `CoreMessagesProvider`, plus the same reflection-based field-count guard so a new `QueryIndex` field not mirrored in `QueryIndexMessage` fails the test (the parity check the QueryEntity test doesn't extend to the nested index). Targeted at `ignite-28767` so you can pull it in here if it looks good. -- 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]
