[
https://issues.apache.org/jira/browse/TAJO-539?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13881703#comment-13881703
]
Hyunsik Choi commented on TAJO-539:
-----------------------------------
Updated the review request against branch master in reviewboard
https://reviews.apache.org/r/17308/
> Change some EvalNode::eval to directly return a Datum value
> -----------------------------------------------------------
>
> Key: TAJO-539
> URL: https://issues.apache.org/jira/browse/TAJO-539
> Project: Tajo
> Issue Type: Improvement
> Components: operator/expression
> Reporter: Hyunsik Choi
> Assignee: Hyunsik Choi
> Fix For: 0.8-incubating
>
> Attachments: TAJO-539.patch, TAJO-539_20140125_15:13:44.patch
>
>
> After TAJO-501, we can ensure that all expressions except for
> AggregationFunctionCallEval can be evaluated by calling once eval() instead
> of calling eval followed by terminate. In addition, current EvalNode
> implementation involves unnecessary memory consumption to keep EvalContext
> for all expressions which are not even aggregation eval.
> If we change EvalNode::eval() to directly return a Datum value, it would be
> reduce the memory consumption and CPU costs.
--
This message was sent by Atlassian JIRA
(v6.1.5#6160)
