anton-vinogradov commented on code in PR #13095:
URL: https://github.com/apache/ignite/pull/13095#discussion_r3588816107
##########
modules/core/src/main/java/org/apache/ignite/internal/processors/cache/StoredCacheData.java:
##########
@@ -45,21 +43,23 @@
* <p>
* All changes must be made with the respect of RU rules.
*/
-public class StoredCacheData implements Serializable, CdcCacheEvent,
MarshallableMessage {
+public class StoredCacheData implements Serializable, CdcCacheEvent, Message {
/** */
private static final long serialVersionUID = 0L;
/** Cache configuration. */
@GridToStringInclude
- private CacheConfiguration<?, ?> ccfg;
+ @Marshalled("ccfgBytes")
+ CacheConfiguration<?, ?> ccfg;
/** Serialized {@link #ccfg}. */
@Order(0)
transient byte[] ccfgBytes;
/** Query entities. */
@GridToStringInclude
- private Collection<QueryEntity> qryEntities;
+ @Marshalled("qryEntitiesBytes")
+ Collection<QueryEntity> qryEntities;
Review Comment:
Reverted — `StoredCacheData` is back to the
`@Marshalled("qryEntitiesBytes")` JDK blob, no `MarshallableMessage` hooks.
Agreed the `QueryEntityMessage` migration belongs in its own PR (and `ccfg`
would want the same treatment once a `CacheConfiguration` DTO exists). Compile
+ checkstyle clean.
--
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]
