Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-13655

Phase out MemoryError and TimeoutError from pex::exceptions

    XMLWordPrintable

    Details

      Description

      Per RFC-448, pex::exceptions::MemoryError and pex::exceptions::TimeoutError should both be removed from the Stack.

      C++ code throwing MemoryError should throw std::bad_alloc instead; Python code should raise the built-in MemoryError (note that pybind11 wraps bad_alloc as MemoryError already, so these two changes are mutually consistent).

        Attachments

          Issue Links

          is triggered by

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-448 Document intended purposes of LSST exceptions

          • Implemented

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            krzys Krzysztof Findeisen added a comment -

            Hi Pim Schellart [X], can you review this removal of two exceptions from the LSST framework?

            Show
            krzys Krzysztof Findeisen added a comment - Hi Pim Schellart [X] , can you review this removal of two exceptions from the LSST framework?
            Hide
            Permalink
            pschella Pim Schellart [X] (Inactive) added a comment -

            Looks good. One question on comment, but feel free to ignore.

            Show
            pschella Pim Schellart [X] (Inactive) added a comment - Looks good. One question on comment, but feel free to ignore.

              People

              Assignee:
              krzys Krzysztof Findeisen
              Reporter:
              krzys Krzysztof Findeisen
              Reviewers:
              Pim Schellart [X] (Inactive)
              Watchers:
              Krzysztof Findeisen, Pim Schellart [X] (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.