fix: do not init sequence when insert lru

Signed-off-by: MrCroxx <[email protected]>
risingwavelabs · Apr 2, 2024 · 3959a16 · 3959a16
1 parent f9678c3
commit 3959a16
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/common/src/lru.rs b/src/common/src/lru.rs
@@ -153,7 +153,8 @@ where
                             key: MaybeUninit::new(key),
                             value: MaybeUninit::new(value),
                             hash,
-                            sequence: SEQUENCER.with(|s| s.borrow_mut().inc()),
+                            // sequence will be updated by `attach`
+                            sequence: 0,
                         },
                         self.alloc.clone(),
                     );

diff --git a/src/common/src/sequence.rs b/src/common/src/sequence.rs
@@ -29,7 +29,7 @@ pub struct Sequencer {
 
 impl Sequencer {
     pub const DEFAULT_LAG: Sequence = Self::DEFAULT_STEP * 16;
-    pub const DEFAULT_STEP: Sequence = 128;
+    pub const DEFAULT_STEP: Sequence = 64;
 
     pub const fn new(step: Sequence, lag: Sequence) -> Self {
         Self {