Uploaded image for project: 'OpenJPA'
  1. OpenJPA
  2. OPENJPA-2006

Sequence generated ids may fail to get assigned when flushing a graph with bi-directional relationships

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 1.2.2, 2.1.0, 2.2.0
    • 1.2.3, 2.0.2, 2.1.2, 2.2.0
    • kernel
    • None

    Description

      Flushing a multi-level graph with many bi-directional relationships can result in SQL exceptions due to sequence-generated IDs of some leaf entities not getting assigned during pre-flush. Sequence-generated id's require an additional ID population step which happens during pre-flush. OpenJPA's state transitioning during a pre-flush can fail to pre-flush child entities when certain bi-directional traversals occur. The proposed fix is if flushing, perform the pre-flush on entities that may have moved from provisional to new state (as a result of a previous traversal) without being properly pre-flushed. If the entity was previously pre-flushed, it will have been tagged as such and the operation will simply return. Otherwise, it'll pre-flush the entity and all child entities, generating IDs in the process. On Oracle, the SQL exception below is a symptom of the problem. An ID has not been assigned to an entity and as a result, OpenJPA is attempting to insert NULL into an ID field.

      Caused by: org.apache.openjpa.lib.jdbc.ReportingSQLException: ORA-01400: cannot insert NULL into ("SCOTT"."FL_ASSIGN"."ASSIGN_ID")

      {prepstmnt 577905266 INSERT INTO FL_ASSIGN (ASSIGN_ID, ASSIGN_TEXT, TOPIC_ID) VALUES (?, ?, ?) [params=?, ?, ?]}

      [code=1400, state=23000]

      Attachments

        1. OPENJPA-2006.1.2.x.patch
          29 kB
          Jeremy Bauer

        Activity

          People

            techhusky Jeremy Bauer
            techhusky Jeremy Bauer
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: