OpenJPA
  1. OpenJPA
  2. OPENJPA-2348

Cache calculated hashCode in OpenJPAId

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Invalid
    • Affects Version/s: 2.2.2, 2.3.0
    • Fix Version/s: 2.2.2, 2.3.0
    • Component/s: kernel
    • Labels:
      None

      Description

      Under many use cases we will calculate and then re-calculate the hashCode value for OpenJPAIds and we could save some processing by caching this value.

        Activity

        Rick Curtis created issue -
        Hide
        ASF subversion and git services added a comment -

        Commit 1452884 from curtisr7
        [ https://svn.apache.org/r1452884 ]

        OPENJPA-2348: Cache calculated hashcode in OpenJPAId.

        Show
        ASF subversion and git services added a comment - Commit 1452884 from curtisr7 [ https://svn.apache.org/r1452884 ] OPENJPA-2348 : Cache calculated hashcode in OpenJPAId.
        Hide
        Rick Curtis added a comment -

        After further analysis, this change provided little, to no benefit so I'm going to revert the previous commit.

        Show
        Rick Curtis added a comment - After further analysis, this change provided little, to no benefit so I'm going to revert the previous commit.
        Hide
        ASF subversion and git services added a comment -

        Commit 1453928 from curtisr7
        [ https://svn.apache.org/r1453928 ]

        OPENJPA-2348: Revert prior commit.

        Show
        ASF subversion and git services added a comment - Commit 1453928 from curtisr7 [ https://svn.apache.org/r1453928 ] OPENJPA-2348 : Revert prior commit.
        Rick Curtis made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Invalid [ 6 ]
        Rick Curtis made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        1d 23h 31m 1 Rick Curtis 07/Mar/13 15:50
        Resolved Resolved Closed Closed
        18d 20m 1 Rick Curtis 25/Mar/13 16:10

          People

          • Assignee:
            Rick Curtis
            Reporter:
            Rick Curtis
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development