4paradigm / 4paradigm/OpenMLDB
long window: incorrect result for pure history window
- Dominant language
- C++
- Stars
- 1.7k
- Forks
- 331
- Avg merge
- 12d 12h
- Merged PRs (30d)
- 1
Description
**Bug Description**
https://github.com/4paradigm/OpenMLDB/blob/b44617e0300fc2f2ca8ed6a8df5bd70d5a8c488d/hybridse/src/vm/runner.cc#L2915
current row always count into window agg even for pure history window
**Expected Behavior**
**Steps to Reproduce**
1.
2.
3.
4.
Contributor guide
Research direction
The bug is in hybridse/src/vm/runner.cc at line 2915, where the current row is incorrectly counted in a pure history window aggregation. Start by examining the window aggregation logic around that line and the definition of a pure history window. Look for existing tests related to window functions to understand the expected behavior. Verify the fix by running relevant unit or integration tests in the codebase.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- databases
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100