Uploaded image for project: 'Tajo'
  1. Tajo
  2. TAJO-539

Change some EvalNode::eval to directly return a Datum value

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.8.0
    • Component/s: Expression
    • Labels:
      None

      Description

      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 reduce the memory consumption and CPU costs.

        Attachments

        1. TAJO-539_20140125_15:13:44.patch
          180 kB
          Hyunsik Choi
        2. TAJO-539.patch
          180 kB
          Hyunsik Choi

          Activity

            People

            • Assignee:
              hyunsik Hyunsik Choi
              Reporter:
              hyunsik Hyunsik Choi
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: