Derby
  1. Derby
  2. DERBY-5062

[patch] push code assignments down to where they are used - part 2

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 10.7.1.1
    • Fix Version/s: 10.8.1.2
    • Component/s: Services
    • Urgency:
      Low
    • Issue & fix info:
      Patch Available
    • Bug behavior facts:
      Performance

      Description

      code does various assignments at a higher level than the assignments are needed, causing code to execute unnecessarily if the expected code path isn't taken. Push these assignments down to where they are actually used.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        5d 7h 13m 1 Dave Brosius 25/Feb/11 01:04
        Resolved Resolved Closed Closed
        1d 6h 33m 1 Dave Brosius 26/Feb/11 07:38
        Gavin made changes -
        Workflow jira [ 12599402 ] Default workflow, editable Closed status [ 12801716 ]
        Rick Hillegas made changes -
        Fix Version/s 10.8.1.1 [ 12316356 ]
        Fix Version/s 10.8.1.2 [ 12316362 ]
        Rick Hillegas made changes -
        Fix Version/s 10.8.1.1 [ 12316356 ]
        Fix Version/s 10.8.1.0 [ 12315561 ]
        Dave Brosius made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Dave Brosius made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Lily Wei made changes -
        Labels derby_triage10_8
        Bug behavior facts [Performance]
        Assignee Dave Brosius [ dbrosius@apache.org ]
        Issue & fix info [Patch Available]
        Hide
        Lily Wei added a comment -

        I've committed with revision 1074227.Thanks, Dave.

        Show
        Lily Wei added a comment - I've committed with revision 1074227.Thanks, Dave.
        Hide
        Lily Wei added a comment -

        Suites.all and derbyall run clean with this patch. If there is no objection, I will commit this patch soon.

        Show
        Lily Wei added a comment - Suites.all and derbyall run clean with this patch. If there is no objection, I will commit this patch soon.
        Dave Brosius made changes -
        Field Original Value New Value
        Attachment push_down_code.diff [ 12471476 ]
        Dave Brosius created issue -

          People

          • Assignee:
            Dave Brosius
            Reporter:
            Dave Brosius
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 1h
              1h
              Remaining:
              Remaining Estimate - 1h
              1h
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development