Skip to content

KYLIN-2820 Query can't read window function's result from subquery#796

Merged
luguosheng1314 merged 1 commit intoapache:masterfrom
nichunen:KYLIN-2820
Aug 11, 2019
Merged

KYLIN-2820 Query can't read window function's result from subquery#796
luguosheng1314 merged 1 commit intoapache:masterfrom
nichunen:KYLIN-2820

Conversation

@nichunen
Copy link

No description provided.

@asfgit
Copy link

asfgit commented Aug 10, 2019

Can one of the admins verify this patch?

@codecov-io
Copy link

codecov-io commented Aug 10, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@6668b0d). Click here to learn what that means.
The diff coverage is 25.8%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #796   +/-   ##
=========================================
  Coverage          ?   25.72%           
  Complexity        ?     6048           
=========================================
  Files             ?     1394           
  Lines             ?    83110           
  Branches          ?    11634           
=========================================
  Hits              ?    21378           
  Misses            ?    59678           
  Partials          ?     2054
Impacted Files Coverage Δ Complexity Δ
.../query/relnode/visitor/TupleExpressionVisitor.java 0% <0%> (ø) 0 <0> (?)
.../org/apache/kylin/query/relnode/OLAPWindowRel.java 0% <0%> (ø) 0 <0> (?)
...org/apache/kylin/query/relnode/OLAPProjectRel.java 0% <0%> (ø) 0 <0> (?)
...g/apache/kylin/query/relnode/OLAPAggregateRel.java 0% <0%> (ø) 0 <0> (?)
.../org/apache/kylin/query/relnode/ColumnRowType.java 0% <0%> (ø) 0 <0> (?)
...ava/org/apache/kylin/metadata/model/TblColRef.java 51.26% <80%> (ø) 25 <3> (?)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6668b0d...e37bfdc. Read the comment docs.

@coveralls
Copy link

coveralls commented Aug 10, 2019

@luguosheng1314 luguosheng1314 merged commit abc0ab8 into apache:master Aug 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Kylin 3.X Kylin 3.X

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants