[
https://issues.apache.org/jira/browse/KYLIN-3579?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16633205#comment-16633205
]
ASF subversion and git services commented on KYLIN-3579:
--------------------------------------------------------
Commit 8db033097833be5118655a6d88a5b96e862055ae in kylin's branch
refs/heads/master from chao long
[ https://gitbox.apache.org/repos/asf?p=kylin.git;h=8db0330 ]
KYLIN-3579 Replace keySet iterator with entrySet iterator
> 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)
