NSAmelchev commented on code in PR #10360:
URL: https://github.com/apache/ignite/pull/10360#discussion_r1018714060
##########
modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/IgniteSnapshotManager.java:
##########
@@ -394,6 +403,13 @@ public class IgniteSnapshotManager extends
GridCacheSharedManagerAdapter
/** Snapshot transfer rate limit in bytes/sec. */
private final DistributedLongProperty snapshotTransferRate =
detachedLongProperty(SNAPSHOT_TRANSFER_RATE_DMS_KEY);
+ /**
+ * Snapshot delta sort batch size in pages count. If set then delta pages
will be indexed by page index to almost
+ * sequential disk write on apply. This generates an extra delta read. If
value is non-positive or not set then delta
+ * pages will be applied directly.
+ */
+ private final DistributedIntegerProperty deltaSortBatch =
detachedIntegerProperty(SNAPSHOT_DELTA_SORT_BATCH_SIZE_KEY);
Review Comment:
> We don't even need to change this parameter at runtime.
The snapshot path is set on each snapshot create process. I think we should
allow users to set a mode per hardware. WDYT?
--
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]
