[
https://issues.apache.org/jira/browse/KYLIN-3635?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16666699#comment-16666699
]
ASF GitHub Bot commented on KYLIN-3635:
---------------------------------------
Aaaaaaron commented on a change in pull request #316: KYLIN-3635, make a
stronger constraint about implementing method reset().
URL: https://github.com/apache/kylin/pull/316#discussion_r228788582
##########
File path:
core-metadata/src/main/java/org/apache/kylin/measure/MeasureIngester.java
##########
@@ -43,9 +43,7 @@
abstract public V valueOf(String[] values, MeasureDesc measureDesc,
Map<TblColRef, Dictionary<String>> dictionaryMap);
- public void reset() {
-
- }
+ abstract public void reset();
Review comment:
Thanks, Shaofeng, Pretty good advice!
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> Percentile calculation on Spark engine is wrong
> -----------------------------------------------
>
> Key: KYLIN-3635
> URL: https://issues.apache.org/jira/browse/KYLIN-3635
> Project: Kylin
> Issue Type: Bug
> Components: Spark Engine
> Affects Versions: v2.3.0, v2.3.1, v2.4.0, v2.3.2, v2.4.1, v2.5.0
> Reporter: Shaofeng SHI
> Assignee: Shaofeng SHI
> Priority: Major
> Fix For: v2.4.2, v2.5.1
>
>
> As titled; Received reporting that percentile result is wrong when using
> Spark engine. Checked the code and found the object was reused and not reset,
> the problem won't happen on normal MR as it would be serialized before next
> call, but would be a problem on Spark, as the object was cached in-memory and
> be serialized later.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
