Skip to content

Commit 8e74198

Browse files
zz_yclaude
andcommitted
bench: rename New* store variants to Current* for clarity
NewPerKey/NewGlobal implied a not-yet-merged implementation; renaming to CurrentPerKey/CurrentGlobal better reflects that these benchmark the current SimpleMapStore dispatch path. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 96268be commit 8e74198

1 file changed

Lines changed: 8 additions & 8 deletions

File tree

asap-query-engine/benches/simple_store_bench.rs

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -42,37 +42,37 @@ use std::sync::{Arc, Barrier};
4242
enum StoreKind {
4343
LegacyPerKey,
4444
LegacyGlobal,
45-
NewPerKey,
46-
NewGlobal,
45+
CurrentPerKey,
46+
CurrentGlobal,
4747
}
4848

4949
impl StoreKind {
5050
const ALL: [Self; 4] = [
5151
Self::LegacyPerKey,
5252
Self::LegacyGlobal,
53-
Self::NewPerKey,
54-
Self::NewGlobal,
53+
Self::CurrentPerKey,
54+
Self::CurrentGlobal,
5555
];
5656

5757
fn slug(self) -> &'static str {
5858
match self {
5959
Self::LegacyPerKey => "legacy/per_key",
6060
Self::LegacyGlobal => "legacy/global",
61-
Self::NewPerKey => "new/per_key",
62-
Self::NewGlobal => "new/global",
61+
Self::CurrentPerKey => "current/per_key",
62+
Self::CurrentGlobal => "current/global",
6363
}
6464
}
6565

6666
fn build(self, config: Arc<StreamingConfig>, cleanup_policy: CleanupPolicy) -> Arc<dyn Store> {
6767
match self {
6868
Self::LegacyPerKey => Arc::new(LegacySimpleMapStorePerKey::new(config, cleanup_policy)),
6969
Self::LegacyGlobal => Arc::new(LegacySimpleMapStoreGlobal::new(config, cleanup_policy)),
70-
Self::NewPerKey => Arc::new(SimpleMapStore::new_with_strategy(
70+
Self::CurrentPerKey => Arc::new(SimpleMapStore::new_with_strategy(
7171
config,
7272
cleanup_policy,
7373
LockStrategy::PerKey,
7474
)),
75-
Self::NewGlobal => Arc::new(SimpleMapStore::new_with_strategy(
75+
Self::CurrentGlobal => Arc::new(SimpleMapStore::new_with_strategy(
7676
config,
7777
cleanup_policy,
7878
LockStrategy::Global,

0 commit comments

Comments
 (0)