[dna-issues] [JBoss JIRA] Updated: (DNA-546) JCR Workspace and Session Imports Can Fail on JPA Connector

[Brian Carothers (JIRA)]

     [ https://jira.jboss.org/jira/browse/DNA-546?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Brian Carothers updated DNA-546:
--------------------------------

    Attachment: DNA-546_jpa_connector_refactor.patch


Attached patch (DNA-546_jpa_connector_refactor.patch) that refactors the existing JPA connector framework to more readily support the new model that I'm working on.

The patch removes the relationship between the model and the request processor and adds a relationship between the model and its connection.  To work within the MapRepository framework, I had to be able to pass a reference to the MapRepository into the MapRequestProcessor.  This ruled out using the existing JpaConnection.  The cleanest way to isolate the reusable pieces of the code seemed to be to rename JpaConnection to BasicJpaConnection, move it into model.basic, and then replace Model.createRequestProcessor(...) with Model.createConnection(JpaSource).  This required a few changes to class visibility (but not constructor visibility) and the addition of some accessors to JpaSource.

Clearly, there are alternate ways to accomplish this same goal.  I am going to wait for some feedback before I commit this patch.


> JCR Workspace and Session Imports Can Fail on JPA Connector
> -----------------------------------------------------------
>
>                 Key: DNA-546
>                 URL: https://jira.jboss.org/jira/browse/DNA-546
>             Project: DNA
>          Issue Type: Bug
>          Components: Connectors, JCR
>    Affects Versions: 0.6
>            Reporter: Brian Carothers
>             Fix For: 0.7
>
>         Attachments: DNA-546_jpa_connector_refactor.patch, DNA-546_map_node_change.patch
>
>
> Importing a set of nodes from an XML file will fail if one of the imported nodes has the same jcr:uuid as an existing node in the graph and removeExisting is true and the new location for the imported node and its prior location are both accessed through the same JPA connector.
> The DocumentViewImportTest is disabled (commented out) in the JPA integration test as a result of this defect.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira