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:
            2 Start watching this issue

              Created:
              Updated: