OpenJPA
  1. OpenJPA
  2. OPENJPA-1366

Remove need for an EMF from OpenJPAPersistenceUtil and TraversableResolverImpl

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.0-M3
    • Fix Version/s: 2.0.0-beta
    • Component/s: validation
    • Labels:
      None

      Description

      As pointed out by Pinaki and confirmed by Jeremy, we don't need an EMF for the isLoaded() call.

      From Pinaki - We can answer isLoaded() without really asking every broker to llok up in its cache or something. If the given instance pc is PersitenceCapable and has a non-null StateManager – it is managed by us. If the question is: is pc managed by a specific EMF e, then also we can navigate from pc->sm->broker->brokerfactory->emf and compare by reference with e.

        Activity

        Donald Woods created issue -
        Donald Woods made changes -
        Field Original Value New Value
        Summary Remove need for an EMF from OpenJPAPersistenceUtil.isLoaded() and TraversableResolverImpl Remove need for an EMF from OpenJPAPersistenceUtil and TraversableResolverImpl
        Donald Woods made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        Donald Woods added a comment -

        committed to trunk as r831406

        Show
        Donald Woods added a comment - committed to trunk as r831406
        Donald Woods made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Donald Woods made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        19h 17m 1 Donald Woods 30/Oct/09 16:21
        In Progress In Progress Resolved Resolved
        1h 41m 1 Donald Woods 30/Oct/09 18:03
        Resolved Resolved Closed Closed
        130d 28m 1 Donald Woods 09/Mar/10 18:31

          People

          • Assignee:
            Donald Woods
            Reporter:
            Donald Woods
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development