Skip to content

Commit

Permalink
moving the comment
Browse files Browse the repository at this point in the history
Signed-off-by: tabokie <xy.tao@outlook.com>
  • Loading branch information
tabokie committed Jun 28, 2023
1 parent 79f3d38 commit dc7719a
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions components/raftstore-v2/src/raft/apply.rs
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,12 @@ pub struct Apply<EK: KvEngine, R> {
// can fetch the wrong apply index from flush_state.
applied_index: u64,
/// The largest index that have modified each column family.
///
/// Caveats: This field must be consistent with the state of memtable. If
/// modified is advanced when memtable is empty, the admin flushed can never
/// be advanced. If modified is not advanced when memtable is written, the
/// corresponding Raft entry may be deleted before the change is fully
/// persisted (flushed).
modifications: DataTrace,
admin_cmd_result: Vec<AdminCmdResult>,
flush_state: Arc<FlushState>,
Expand Down Expand Up @@ -281,11 +287,6 @@ impl<EK: KvEngine, R> Apply<EK, R> {
}
}

/// Caveats: This field must be consistent with the state of memtable. If
/// modified is advanced when memtable is empty, the admin flushed can never
/// be advanced. If modified is not advanced when memtable is written, the
/// corresponding Raft entry may be deleted before the change is fully
/// persisted (flushed).
#[inline]
pub fn modifications_mut(&mut self) -> &mut DataTrace {
&mut self.modifications
Expand Down

0 comments on commit dc7719a

Please sign in to comment.