Support newly added node install and resume snapshot by Linary · Pull Request #1439 · apache/hugegraph
| * Don't perform decompression, it's possible to trigger the bug of | ||
| * IOUtils.skip() infinite loop. I don't know how this bug is | ||
| * generated yet. | ||
| */ |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
where is the snapshot data from
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
from linked directory. The decompressed file is just manifest.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the snapshot recovery process when add newly raft node?
Linary
changed the title
Remove decompressSnapshot operation when resume snapshot
Support newly added node install and resume snapshot
| */ | ||
| this.compressJraftSnapshotDir(writer, jraftSnapshotPath, done); | ||
| this.compressSnapshotDir(writer, snapshotDirMaps, done); | ||
| this.deleteSnapshotDirs(snapshotDirMaps.keySet()); |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
catch compressSnapshotDir and deleteSnapshotDirs
javeme
deleted the
remove-decompress
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters