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

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

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 0.8.0
    • Expression
    • 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

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

            Dates

              Created:
              Updated:
              Resolved: