Make exec::agg::buildPipeline return by value

    • Type: Task
    • Resolution: Unresolved
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: None
    • Query Execution
    • None
    • 3
    • TBD
    • None
    • None
    • None
    • None
    • None
    • None
    • None
    • 0

      Currently exec::agg::buildPipeline returns exec::agg::Pipeline by the means of std::unique_ptr.

      There is no specific reasons on the 'returning' side to return std::unique_ptr, so we can switch to returning by value, as generally recommended.

      So far, the default copy constructor is explicitly deleted, and this is desired to avoid accidental copying. The exec::agg::buildPipeline only needs to be moved, unless we have some new use-cases.

              Assignee:
              Unassigned
              Reporter:
              Romans Kasperovics
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated: