[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #981: IGNITE-17225 Page replacement for persistent data region is not fully ported

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

tkalkirill commented on code in PR #981:
URL: https://github.com/apache/ignite-3/pull/981#discussion_r940072663


##########
modules/page-memory/src/main/java/org/apache/ignite/internal/pagememory/persistence/replacement/DelayedDirtyPageWrite.java:
##########
@@ -44,13 +46,10 @@ public class DelayedDirtyPageWrite implements WriteDirtyPage {
     /** Replacing pages tracker, used to register & unregister pages being written. */
     private final DelayedPageReplacementTracker tracker;
 
-    /** Full page id to be written on {@link #finishReplacement} or null if nothing to write. */
+    /** Full page id to be written on {@link #finishReplacement} or {@code null} if nothing to write. */
     private @Nullable FullPageId fullPageId;
 
-    /** Byte buffer with page data to be written on {@link #finishReplacement} or null if nothing to write. */
-    private @Nullable ByteBuffer byteBuf;
-
-    /** Partition update tag to be used in{@link #finishReplacement} or null if nothing to write. */
+    /** Partition update tag to be used in{@link #finishReplacement} or {@code null} if nothing to write. */

Review Comment:
   fix it



-- 
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