You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@kylin.apache.org by "Chao Long (JIRA)" <ji...@apache.org> on 2018/09/24 09:13:00 UTC
[jira] [Commented] (KYLIN-3579) entrySet iterator should be used in
BPUSCalculator
[ https://issues.apache.org/jira/browse/KYLIN-3579?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16625566#comment-16625566 ]
Chao Long commented on KYLIN-3579:
----------------------------------
I would like to fix this issue.
> entrySet iterator should be used in BPUSCalculator
> ---------------------------------------------------
>
> Key: KYLIN-3579
> URL: https://issues.apache.org/jira/browse/KYLIN-3579
> Project: Kylin
> Issue Type: Bug
> Reporter: Ted Yu
> Priority: Minor
>
> BPUSCalculator.calculateBenefitTotal and BPUSCalculator.initCuboidAggCostMap use keySet iterator where entrySet iterator should be used.
> {code}
> for (Long cuboid : cuboidAggCostMapCopy.keySet()) {
> if (cuboidAggCostMapCopy.get(cuboid) < processCuboidAggCostMap.get(cuboid)) {
> {code}
> CubeCodeSystem.newMetricsAggregators has similar issue.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)