StarRocks / starrocks

StarRocks, a Linux Foundation project, is a next-generation sub-second MPP OLAP database for full analytics scenarios, including multi-dimensional analytics, real-time analytics, and ad-hoc queries.
https://starrocks.io
Apache License 2.0
8.29k stars 1.68k forks source link

[Enhancement] support user variable in analytic function (backport #47728) #47790

Closed mergify[bot] closed 2 days ago

mergify[bot] commented 2 days ago

Why I'm doing:

What I'm doing:

consider user varibale in constant expr check.

Fixes #47225

What type of PR is this:

Does this PR entail a change in behavior?

If yes, please specify the type of change:

Checklist:

Bugfix cherry-pick branch check:

What I'm doing:

consider user varibale in constant expr check.

Fixes #47225

What type of PR is this:

Does this PR entail a change in behavior?

If yes, please specify the type of change:

Checklist:

mergify[bot] commented 2 days ago

Cherry-pick of 09d8d3a93a042c7a264cd6a650143c1b956c14f2 has failed:

On branch mergify/bp/branch-3.2/pr-47728
Your branch is up to date with 'origin/branch-3.2'.

You are currently cherry-picking commit 09d8d3a93a.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
    modified:   fe/fe-core/src/main/java/com/starrocks/analysis/Expr.java
    modified:   fe/fe-core/src/main/java/com/starrocks/analysis/FunctionCallExpr.java
    modified:   fe/fe-core/src/main/java/com/starrocks/analysis/UserVariableExpr.java
    deleted:    fe/fe-core/src/main/java/com/starrocks/common/util/ExprUtil.java
    modified:   fe/fe-core/src/main/java/com/starrocks/sql/analyzer/AnalyticAnalyzer.java
    modified:   fe/fe-core/src/main/java/com/starrocks/sql/analyzer/ExpressionAnalyzer.java
    modified:   fe/fe-core/src/main/java/com/starrocks/sql/analyzer/FunctionAnalyzer.java
    modified:   fe/fe-core/src/main/java/com/starrocks/sql/optimizer/transformer/SqlToScalarOperatorTranslator.java
    modified:   fe/fe-core/src/test/java/com/starrocks/sql/plan/SelectHintTest.java
    modified:   fe/fe-core/src/test/java/com/starrocks/sql/plan/WindowTest.java

Unmerged paths:
  (use "git add <file>..." to mark resolution)
    both modified:   test/sql/test_hint/R/test_hint
    both modified:   test/sql/test_hint/T/test_hint

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

mergify[bot] commented 2 days ago

@mergify[bot]: Backport conflict, please reslove the conflict and resubmit the pr