-
Notifications
You must be signed in to change notification settings - Fork 158
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Doc: update change log for 0.7.2..0.7.3
- Loading branch information
1 parent
8b3f6a4
commit 2c06dc4
Showing
2 changed files
with
43 additions
and
0 deletions.
There are no files selected for viewing
This file contains 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
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
### Changed: | ||
|
||
- Changed: [25e94c36](https://github.com/datafuselabs/openraft/commit/25e94c36e5c8ae640044196070f9a067d5f105a3) InstallSnapshotResponse: replies the last applied log id; Do not install a smaller snapshot; by 张炎泼; 2022-09-22 | ||
|
||
A snapshot may not be installed by a follower if it already has a higher | ||
`last_applied` log id locally. | ||
In such a case, it just ignores the snapshot and respond with its local | ||
`last_applied` log id. | ||
|
||
This way the applied state(i.e., `last_applied`) will never revert back. | ||
|
||
### Fixed: | ||
|
||
- Fixed: [21684bbd](https://github.com/datafuselabs/openraft/commit/21684bbdfdc54b18daa68f623afc2b0be6718c72) potential inconsistency when installing snapshot; by 张炎泼; 2022-09-22 | ||
|
||
The conflicting logs that are before `snapshot_meta.last_log_id` should | ||
be deleted before installing a snapshot. | ||
|
||
Otherwise there is chance the snapshot is installed but conflicting logs | ||
are left in the store, when a node crashes. |