Consider removing std::terminate() call in Pipeline::dispose()

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

      Currently, Pipeline::dispose() method contains the following block

      try { ... }
      catch (...) {
              std::terminate();
      }
      

      Which hides all exceptions and kill the process upon a query, which does not feel very safe (perhaps I am wrong).

      I am adding this ticket to SPM-4106, but this improvement is clearly optional for the project.

      This ticket will become obsolete / solved if we remove all the 'dispose()' methods before.

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

              Created:
              Updated: