Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: OpenCMIS 0.3.0
    • Component/s: chemistry-old
    • Labels:
      None

      Description

      I suggest to port the jcr server code (https://svn.apache.org/repos/asf/chemistry/old/chemistry/trunk/chemistry-jcr/) to the new openCmis code base.

      1. CMIS-315.patch
        141 kB
        Michael Dürig

        Activity

        Hide
        Michael Dürig added a comment -

        Proposed patch for getting started.

        This patch enables CRUD functionality for JCR nodes exposing them as cmis:folder and cmis:files.

        Show
        Michael Dürig added a comment - Proposed patch for getting started. This patch enables CRUD functionality for JCR nodes exposing them as cmis:folder and cmis:files.
        Hide
        Michael Dürig added a comment -

        Above patch adds a chemistry-opencmis-server-jcr module to chemistry-opencmis-server/. It only introduces a mandatory dependency to the JCR api (javax.jcr).

        It has some non mandatory dependencies to Apache Jackrabbit via the 'develop' profile. This makes building and deploying a standalone server for testing purposes easy:

        mvn -Pdevelop clean package jetty:run

        Creates a Jackrabbit JCR repository at

        {user.home}

        /jcr-repository and exposes it on http://localhost:8080/chemistry.opencmis-server-jcr/atom

        Show
        Michael Dürig added a comment - Above patch adds a chemistry-opencmis-server-jcr module to chemistry-opencmis-server/. It only introduces a mandatory dependency to the JCR api (javax.jcr). It has some non mandatory dependencies to Apache Jackrabbit via the 'develop' profile. This makes building and deploying a standalone server for testing purposes easy: mvn -Pdevelop clean package jetty:run Creates a Jackrabbit JCR repository at {user.home} /jcr-repository and exposes it on http://localhost:8080/chemistry.opencmis-server-jcr/atom
        Hide
        Jukka Zitting added a comment -

        Looks pretty good! Unless someone beats me to it or has concerns about this patch, I'd like to transition from my earlier mentor role to a committer and take care of committing this.

        Show
        Jukka Zitting added a comment - Looks pretty good! Unless someone beats me to it or has concerns about this patch, I'd like to transition from my earlier mentor role to a committer and take care of committing this.
        Hide
        Florian Müller added a comment -

        Looks good to me, too. +1 for commit.

        Show
        Florian Müller added a comment - Looks good to me, too. +1 for commit.
        Hide
        Jukka Zitting added a comment -

        Patch committed in revision 1073887. Leaving the issue open until this rough first version has been further refined.

        Show
        Jukka Zitting added a comment - Patch committed in revision 1073887. Leaving the issue open until this rough first version has been further refined.
        Hide
        Michael Dürig added a comment -

        Thanks for applying the patch. The reactor pom didn't make it though. Is this by intention?

        Index: pom.xml
        ===================================================================
        — pom.xml (revision 1065547)
        +++ pom.xml (revision )
        @@ -168,6 +168,7 @@
        <module>chemistry-opencmis-server/chemistry-opencmis-server-support</module>
        <module>chemistry-opencmis-server/chemistry-opencmis-server-inmemory</module>
        <module>chemistry-opencmis-server/chemistry-opencmis-server-fileshare</module>
        + <module>chemistry-opencmis-server/chemistry-opencmis-server-jcr</module>
        <module>chemistry-opencmis-test/chemistry-opencmis-test-fit</module>
        <module>chemistry-opencmis-test/chemistry-opencmis-test-tck</module>
        <module>chemistry-opencmis-test/chemistry-opencmis-test-tools</module>

        Show
        Michael Dürig added a comment - Thanks for applying the patch. The reactor pom didn't make it though. Is this by intention? Index: pom.xml =================================================================== — pom.xml (revision 1065547) +++ pom.xml (revision ) @@ -168,6 +168,7 @@ <module>chemistry-opencmis-server/chemistry-opencmis-server-support</module> <module>chemistry-opencmis-server/chemistry-opencmis-server-inmemory</module> <module>chemistry-opencmis-server/chemistry-opencmis-server-fileshare</module> + <module>chemistry-opencmis-server/chemistry-opencmis-server-jcr</module> <module>chemistry-opencmis-test/chemistry-opencmis-test-fit</module> <module>chemistry-opencmis-test/chemistry-opencmis-test-tck</module> <module>chemistry-opencmis-test/chemistry-opencmis-test-tools</module>
        Hide
        Jukka Zitting added a comment -

        Sorry about that, I committed the changes only from within the server subdirectory. The reactor change is now in, starting with revision 1074533.

        Show
        Jukka Zitting added a comment - Sorry about that, I committed the changes only from within the server subdirectory. The reactor change is now in, starting with revision 1074533.
        Hide
        Jukka Zitting added a comment -

        Resolving as fixed. See followup issues like CMIS-329 for additional work in this area.

        Show
        Jukka Zitting added a comment - Resolving as fixed. See followup issues like CMIS-329 for additional work in this area.

          People

          • Assignee:
            Jukka Zitting
            Reporter:
            Michael Dürig
          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development