[GitHub] [kylin] Qsimple commented on issue #511: KYLIN-3830 return wrong result when 'SELECT SUM(dim1)' without set a …

[GitBox <gi...@apache.org>]

Qsimple commented on issue #511: KYLIN-3830 return wrong result when 'SELECT SUM(dim1)' without set a …
URL: https://github.com/apache/kylin/pull/511#issuecomment-471965880
 
 
   Yes, I have tested it. when query by `SELECT SUM(SELLER_ID) FROM KYLIN_SALES;`, kylin throw
   ```
   No realization found for OLAPContext, CUBE_NOT_READY, CUBE_NOT_READY, CUBE_UNMATCHED_AGGREGATION[FunctionDesc [expression=SUM, parameter=1_317b3789:DEFAULT.KYLIN_SALES.SELLER_ID, returnType=null]], CUBE_NOT_READY, CUBE_NOT_READY, CUBE_NOT_READY, CUBE_NOT_READY, CUBE_NOT_READY, CUBE_NOT_READY, CUBE_NOT_READY, rel#59:OLAPTableScan.OLAP.[](table=[DEFAULT, KYLIN_SALES],ctx=,fields=[0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12]) while executing SQL: "SELECT SUM(SELLER_ID) FROM KYLIN_SALES LIMIT 50000"
   ```   
   Because I have found `FunctionDesc.BUILT_IN_AGGREGATIONS` only used in CubeCapabilityChecker:tryDimensionAsMeasure:235, which judge the FunctionDesc(SUM,COUNT,MAX,MIN,PERCENTILE,COUNT_DISTINCT) can be used as Dimension as measure.  
   As we discussed before, the SUM can't be calculated right. So I remove it to reject such query. Although the error message seems obscure, the commit can fix this bug with least change. 
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services