Skip to content

Commit 99d0363

Browse files
committed
allow range frame calls using range cache policies other than full
Signed-off-by: Richard Chien <[email protected]>
1 parent 199be83 commit 99d0363

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

src/stream/src/executor/over_window/general.rs

+1-3
Original file line numberDiff line numberDiff line change
@@ -171,9 +171,7 @@ impl<S: StateStore> OverWindowExecutor<S> {
171171
.calls
172172
.iter()
173173
.any(|call| call.frame.bounds.is_unbounded());
174-
// TODO(rc): only support full cache for `RANGE` frame for now, for the sake of simplicity
175-
let has_range_frame = args.calls.iter().any(|call| call.frame.bounds.is_range());
176-
let cache_policy = if has_unbounded_frame || has_range_frame {
174+
let cache_policy = if has_unbounded_frame {
177175
// For unbounded frames, we finally need all entries of the partition in the cache,
178176
// so for simplicity we just use full cache policy for these cases.
179177
CachePolicy::Full

0 commit comments

Comments
 (0)