4paradigm / 4paradigm/OpenMLDB

long window: incorrect result for pure history window

Open
#2,163 0 comments 0 reactions 1 assignee Claimed by @aceforeverd View on GitHub
bug execute-engine
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.