[
https://issues.apache.org/jira/browse/IGNITE-16150?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Amelchev Nikita updated IGNITE-16150:
-------------------------------------
Ignite Flags: Release Notes Required
> Get rid of redundant copying of files after downloading when restoring a
> snapshot.
> -----------------------------------------------------------------------------------
>
> Key: IGNITE-16150
> URL: https://issues.apache.org/jira/browse/IGNITE-16150
> Project: Ignite
> Issue Type: Bug
> Affects Versions: 2.12
> Reporter: Pavel Pereslegin
> Assignee: Pavel Pereslegin
> Priority: Major
> Labels: iep-43
> Fix For: 2.13
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> During the snapshot restore procedure after retrieving the partition file
> from the remote node, we create an additional copy of this file.
> {code:java}
> for (Map.Entry<UUID, Map<Integer, Set<Integer>>> m : snpAff.entrySet())
> {
> ctx.cache().context().snapshotMgr()
> .requestRemoteSnapshotFiles(m.getKey(),
> opCtx0.snpName,
> m.getValue(),
> opCtx0.stopChecker,
> (snpFile, t) -> {
> ...
> Path partFile =
> Paths.get(tmpCacheDir.getAbsolutePath(), snpFile.getName());
> try {
>
> IgniteSnapshotManager.copy(snpMgr.ioFactory(),
> snpFile,
> partFile.toFile(),
> snpFile.length());
> ...
> {code}
> File copying is redundant here and can have significant performance overhead.
> Instead, we have to download the file to the target directory (and rename it
> to the desired name if necessary).
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
