Skip to content

Transform WindowNode and other nodes into ChangePointNode#17246

Open
Sh-Zh-7 wants to merge 5 commits intomasterfrom
perf/szh/change_point_in_window
Open

Transform WindowNode and other nodes into ChangePointNode#17246
Sh-Zh-7 wants to merge 5 commits intomasterfrom
perf/szh/change_point_in_window

Conversation

@Sh-Zh-7
Copy link
Contributor

@Sh-Zh-7 Sh-Zh-7 commented Mar 3, 2026

This PR merges FilterNode -> WindowNode -> TableScanNode into specific ChangePointNode which uses TsFile statistics to accelerate.

@Sh-Zh-7 Sh-Zh-7 marked this pull request as draft March 3, 2026 07:58
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 3, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
5.0% Duplication on New Code (required ≤ 5%)

See analysis details on SonarQube Cloud

@codecov
Copy link

codecov bot commented Mar 3, 2026

Codecov Report

❌ Patch coverage is 37.33075% with 324 lines in your changes missing coverage. Please review.
✅ Project coverage is 39.71%. Comparing base (7e0837b) to head (ad1e94f).
⚠️ Report is 16 commits behind head on master.

Files with missing lines Patch % Lines
...execution/operator/source/ChangePointOperator.java 46.92% 138 Missing ⚠️
...ion/operator/process/TableChangePointOperator.java 0.00% 117 Missing ⚠️
.../plan/relational/planner/node/ChangePointNode.java 41.02% 23 Missing ⚠️
...e/rule/ReplaceFilterWindowLeadWithChangePoint.java 72.22% 20 Missing ⚠️
...eryengine/plan/planner/TableOperatorGenerator.java 0.00% 17 Missing ⚠️
...nner/distribute/TableDistributedPlanGenerator.java 0.00% 7 Missing ⚠️
...eryengine/plan/planner/plan/node/PlanNodeType.java 50.00% 1 Missing ⚠️
...yengine/plan/relational/planner/node/Patterns.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #17246      +/-   ##
============================================
+ Coverage     39.51%   39.71%   +0.20%     
  Complexity      282      282              
============================================
  Files          5101     5106       +5     
  Lines        341895   342501     +606     
  Branches      43525    43657     +132     
============================================
+ Hits         135085   136012     +927     
+ Misses       206810   206489     -321     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Sh-Zh-7 Sh-Zh-7 marked this pull request as ready for review March 5, 2026 03:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant