sashapolo commented on code in PR #10118:
URL: https://github.com/apache/ignite/pull/10118#discussion_r910929286
##########
modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/wal/serializer/RecordDataV1Serializer.java:
##########
@@ -210,22 +208,15 @@ public RecordDataV1Serializer(GridCacheSharedContext
cctx) {
@Override public WALRecord readRecord(RecordType type,
ByteBufferBackedDataInput in, int size)
throws IOException, IgniteCheckedException {
if (type == ENCRYPTED_RECORD || type == ENCRYPTED_RECORD_V2) {
- if (encSpi == null) {
- T2<Integer, RecordType> knownData = skipEncryptedRecord(in,
true);
+ EncryptedData encryptedData = readEncryptedData(in, true, type ==
ENCRYPTED_RECORD_V2);
Review Comment:
I don't think I fully understand your suggestion: `ENCRYPTED_RECORD` and
`ENCRYPTED_RECORD_V2` are not serialized solely by corresponding serializers,
but rather by both of them.
--
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]
