You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@kylin.apache.org by "Shaofeng SHI (JIRA)" <ji...@apache.org> on 2018/06/15 06:35:00 UTC
[jira] [Resolved] (KYLIN-3386) the TopN measure validate code
refactor to make it more clear
[ https://issues.apache.org/jira/browse/KYLIN-3386?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Shaofeng SHI resolved KYLIN-3386.
---------------------------------
Resolution: Fixed
The patch didn't remove three unused import in FunctionRule.java. I merged it and modified the commit. Thanks Yifei.
> the TopN measure validate code refactor to make it more clear
> -------------------------------------------------------------
>
> Key: KYLIN-3386
> URL: https://issues.apache.org/jira/browse/KYLIN-3386
> Project: Kylin
> Issue Type: Improvement
> Reporter: Yifei Wu
> Assignee: Yifei Wu
> Priority: Minor
> Fix For: v2.4.0
>
> Attachments: 0001-KYLIN-3386-adjust-TopN-measure-validation-code.patch
>
>
> _Nowadays, the FunctionRule used for validating measure info, and it should call measureType.validate() code rather than add the logical in this 2 places_
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)