Derby
  1. Derby
  2. DERBY-4443

Wrap rollback in exception handlers in try-catch

    Details

    • Urgency:
      Low
    • Issue & fix info:
      Newcomer

      Description

      Avoid this pattern everywhere:

      }catch(SQLException se)

      { //issue a rollback on any errors conn.rollback(); throw se; }

      because an error in rollback will shadow the original exception.

      1. DERBY-4443.patch
        3 kB
        Houx Zhang
      2. DERBY-4443-1.patch
        7 kB
        Houx Zhang
      3. DERBY-4443-2.patch
        7 kB
        Houx Zhang
      4. DERBY-4443-3.patch
        8 kB
        Houx Zhang
      5. DERBY-4443-4.patch
        10 kB
        Houx Zhang
      6. DERBY-4443-4.png
        15 kB
        Houx Zhang
      7. DERBY-4443-5.patch
        10 kB
        Houx Zhang
      8. DERBY-4443-6-tmp.patch
        10 kB
        Houx Zhang
      9. derby-4443-7.patch
        10 kB
        Houx Zhang

        Issue Links

          Activity

          Gavin made changes -
          Workflow jira [ 12482133 ] Default workflow, editable Closed status [ 12796698 ]
          Myrna van Lunteren made changes -
          Status Reopened [ 4 ] Closed [ 6 ]
          Assignee Myrna van Lunteren [ myrna ] Houx Zhang [ houxzhang ]
          Fix Version/s 10.5.3.2 [ 12315436 ]
          Fix Version/s 10.6.2.3 [ 12315434 ]
          Fix Version/s 10.7.1.4 [ 12315902 ]
          Resolution Fixed [ 1 ]
          Myrna van Lunteren made changes -
          Assignee Houx Zhang [ houxzhang ] Myrna van Lunteren [ myrna ]
          Kathey Marsden made changes -
          Resolution Fixed [ 1 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          Kathey Marsden made changes -
          Link This issue is required by DERBY-5654 [ DERBY-5654 ]
          Myrna van Lunteren made changes -
          Fix Version/s 10.8.2.2 [ 12317968 ]
          Fix Version/s 10.8.2.0 [ 12317955 ]
          Myrna van Lunteren made changes -
          Fix Version/s 10.8.2.0 [ 12317955 ]
          Fix Version/s 10.8.1.6 [ 12316676 ]
          Myrna van Lunteren made changes -
          Status Reopened [ 4 ] Closed [ 6 ]
          Fix Version/s 10.8.1.6 [ 12316676 ]
          Resolution Fixed [ 1 ]
          Myrna van Lunteren made changes -
          Resolution Fixed [ 1 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          Myrna van Lunteren made changes -
          Status Reopened [ 4 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          Myrna van Lunteren made changes -
          Resolution Fixed [ 1 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          Houx Zhang made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Bryan Pendleton made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Issue & fix info [Patch Available, Newcomer] [Newcomer]
          Fix Version/s 10.9.0.0 [ 12316344 ]
          Resolution Fixed [ 1 ]
          Houx Zhang made changes -
          Issue & fix info [Newcomer] [Newcomer, Patch Available]
          Houx Zhang made changes -
          Attachment derby-4443-7.patch [ 12477004 ]
          Houx Zhang made changes -
          Attachment DERBY-4443-6-tmp.patch [ 12476900 ]
          Houx Zhang made changes -
          Attachment DERBY-4443-5.patch [ 12474887 ]
          Houx Zhang made changes -
          Attachment DERBY-4443-4.png [ 12474719 ]
          Houx Zhang made changes -
          Assignee Houx Zhang [ houxzhang ]
          Houx Zhang made changes -
          Attachment DERBY-4443-4.patch [ 12474703 ]
          Houx Zhang made changes -
          Attachment DERBY-4443-3.patch [ 12474519 ]
          Houx Zhang made changes -
          Attachment DERBY-4443-2.patch [ 12474393 ]
          Houx Zhang made changes -
          Attachment DERBY-4443-1.patch [ 12474112 ]
          Houx Zhang made changes -
          Attachment DERBY-4443.patch [ 12472768 ]
          Kathey Marsden made changes -
          Labels derby_triage10_8
          Urgency Low
          Issue & fix info [Newcomer]
          Mike Matrigali made changes -
          Field Original Value New Value
          Component/s Store [ 11412 ]
          Aaron Digulla created issue -

            People

            • Assignee:
              Houx Zhang
              Reporter:
              Aaron Digulla
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development