[GitHub] [kylin] zzcclp commented on pull request #1526: KYLIN-4658 Fix union all issue with regarding to windows function & aggregation on

[GitBox]

zzcclp commented on pull request #1526:
URL: https://github.com/apache/kylin/pull/1526#issuecomment-752824642


   Verified passed. LGTM.
   
   ## Before this pr
   ### Query with window function:
   
![image](https://user-images.githubusercontent.com/9430290/103391782-0c443780-4b56-11eb-9443-3b4a838140f3.png)
   **The result is wrong.**
   
   ### Union query:
   
![image](https://user-images.githubusercontent.com/9430290/103391801-1a925380-4b56-11eb-8bee-778e67c676a8.png)
   
   ## After this pr
   ### Query with window function:
   
![image](https://user-images.githubusercontent.com/9430290/103391814-29790600-4b56-11eb-9a4e-30138f046189.png)
   The result is right.
   
   ### Union query:
   
![image](https://user-images.githubusercontent.com/9430290/103391835-3e559980-4b56-11eb-80fd-20c2c5adb9fb.png)
   
   
   
   
   
   
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org