Mahout
  1. Mahout
  2. MAHOUT-699

Rename taste-webapp module to integration; move integration code there from examples

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.5
    • Fix Version/s: 0.6
    • Labels:
      None

      Description

      Per discussion on the mailing list we need a more rationalized approach to "integration" code of the sort that lives in bits in examples, in taste-webapp, and of the sort mentioned on the mailing list in the context of MongoDB.

        Activity

        Sean Owen made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Hide
        Hudson added a comment -

        Integrated in Mahout-Quality #855 (See https://builds.apache.org/hudson/job/Mahout-Quality/855/)
        MAHOUT-699 Make utils, taste-webapp into 'integration', for optional components of the library which at this point is mostly third-party integration stuff. Remove war packaging. Move JDBC code to integration from core

        srowen : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1131341
        Files :

        • /mahout/trunk/integration/src/test/java/org/apache/mahout/cf/taste
        • /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/model
        • /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/recommender
        • /mahout/trunk/integration/src/test/java/org/apache/mahout/cf/taste/impl/similarity/jdbc
        • /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/similarity/jdbc/AbstractJDBCInMemoryItemSimilarity.java
        • /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/similarity/jdbc/MySQLJDBCInMemoryItemSimilarity.java
        • /mahout/trunk/distribution/src/main/assembly/bin.xml
        • /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/model/jdbc
        • /mahout/trunk/integration/bin
        • /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/similarity
        • /mahout/trunk/distribution/pom.xml
        • /mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/recommender/slopeone/jdbc
        • /mahout/trunk/examples/pom.xml
        • /mahout/trunk/integration
        • /mahout/trunk/integration/src/main/java/org/apache/mahout/utils
        • /mahout/trunk/.gitignore
        • /mahout/trunk/taste-web
        • /mahout/trunk/integration/src/main/java/org/apache/mahout/benchmark
        • /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/recommender/slopeone/jdbc
        • /mahout/trunk/bin/mahout
        • /mahout/trunk/integration/src/test/java/org/apache/mahout/cf/taste/impl/similarity
        • /mahout/trunk/integration/src/main/java/org/apache/mahout/clustering
        • /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl
        • /mahout/trunk/core/pom.xml
        • /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/recommender/slopeone
        • /mahout/trunk/integration/src/main/java/org/apache/mahout/text
        • /mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/similarity/jdbc
        • /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/similarity/jdbc/MySQLJDBCItemSimilarity.java
        • /mahout/trunk/integration/src/test/java/org
        • /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/similarity/jdbc/AbstractJDBCItemSimilarity.java
        • /mahout/trunk/integration/src/test/java/org/apache/mahout/cf/taste/impl/similarity/jdbc/MySQLJDBCInMemoryItemSimilarityTest.java
        • /mahout/trunk/integration/src/test/java/org/apache/mahout/cf/taste/impl
        • /mahout/trunk/utils
        • /mahout/trunk/core/src/test/java/org/apache/mahout/cf/taste/impl/similarity/jdbc
        • /mahout/trunk/pom.xml
        • /mahout/trunk/integration/src/test/java/org/apache/mahout/cf
        • /mahout/trunk/integration/pom.xml
        • /mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/jdbc
        • /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/similarity/jdbc
        Show
        Hudson added a comment - Integrated in Mahout-Quality #855 (See https://builds.apache.org/hudson/job/Mahout-Quality/855/ ) MAHOUT-699 Make utils, taste-webapp into 'integration', for optional components of the library which at this point is mostly third-party integration stuff. Remove war packaging. Move JDBC code to integration from core srowen : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1131341 Files : /mahout/trunk/integration/src/test/java/org/apache/mahout/cf/taste /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/model /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/recommender /mahout/trunk/integration/src/test/java/org/apache/mahout/cf/taste/impl/similarity/jdbc /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/similarity/jdbc/AbstractJDBCInMemoryItemSimilarity.java /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/similarity/jdbc/MySQLJDBCInMemoryItemSimilarity.java /mahout/trunk/distribution/src/main/assembly/bin.xml /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/model/jdbc /mahout/trunk/integration/bin /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/similarity /mahout/trunk/distribution/pom.xml /mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/recommender/slopeone/jdbc /mahout/trunk/examples/pom.xml /mahout/trunk/integration /mahout/trunk/integration/src/main/java/org/apache/mahout/utils /mahout/trunk/.gitignore /mahout/trunk/taste-web /mahout/trunk/integration/src/main/java/org/apache/mahout/benchmark /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/recommender/slopeone/jdbc /mahout/trunk/bin/mahout /mahout/trunk/integration/src/test/java/org/apache/mahout/cf/taste/impl/similarity /mahout/trunk/integration/src/main/java/org/apache/mahout/clustering /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl /mahout/trunk/core/pom.xml /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/recommender/slopeone /mahout/trunk/integration/src/main/java/org/apache/mahout/text /mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/similarity/jdbc /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/similarity/jdbc/MySQLJDBCItemSimilarity.java /mahout/trunk/integration/src/test/java/org /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/similarity/jdbc/AbstractJDBCItemSimilarity.java /mahout/trunk/integration/src/test/java/org/apache/mahout/cf/taste/impl/similarity/jdbc/MySQLJDBCInMemoryItemSimilarityTest.java /mahout/trunk/integration/src/test/java/org/apache/mahout/cf/taste/impl /mahout/trunk/utils /mahout/trunk/core/src/test/java/org/apache/mahout/cf/taste/impl/similarity/jdbc /mahout/trunk/pom.xml /mahout/trunk/integration/src/test/java/org/apache/mahout/cf /mahout/trunk/integration/pom.xml /mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/jdbc /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/similarity/jdbc
        Sean Owen made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Sean Owen added a comment -

        Done. The most substantive change I think anyone will see/care about is a rename: utils is now integration. Otherwise, "my" old JDBC and Axis code has just gone away or been shoved away from core.

        Show
        Sean Owen added a comment - Done. The most substantive change I think anyone will see/care about is a rename: utils is now integration. Otherwise, "my" old JDBC and Axis code has just gone away or been shoved away from core.
        Hide
        Sean Owen added a comment -

        So the totality of what I propose now is:

        • Make taste-webapp into "integration"
        • Remove the .war packaging and Axis dependency (not removing support, just the special-case handling of building a .war file – nice trick and all but a bit at odds with where Mahout is now)
        • Move JDBC-related code and dependency from core to integration
        • Put new MongoDB integration in this module
        • Push utils into this integration module and remove it as a separate one
        Show
        Sean Owen added a comment - So the totality of what I propose now is: Make taste-webapp into "integration" Remove the .war packaging and Axis dependency (not removing support, just the special-case handling of building a .war file – nice trick and all but a bit at odds with where Mahout is now) Move JDBC-related code and dependency from core to integration Put new MongoDB integration in this module Push utils into this integration module and remove it as a separate one
        Hide
        Sean Owen added a comment -

        Question: what do we even think about rolling mahout-utils into said mahout-integration module?

        There's no harm in keeping them separate if they truly serve different purposes. I am not sure if they do now; both seem to have stuff that connects stuff in core/ to other systems, formats, containers, etc. (And a few things in there are arguably examples/.)

        I am mildly in favor of this on grounds of simplification, but am open to thoughts here?

        Show
        Sean Owen added a comment - Question: what do we even think about rolling mahout-utils into said mahout-integration module? There's no harm in keeping them separate if they truly serve different purposes. I am not sure if they do now; both seem to have stuff that connects stuff in core/ to other systems, formats, containers, etc. (And a few things in there are arguably examples/.) I am mildly in favor of this on grounds of simplification, but am open to thoughts here?
        Sean Owen created issue -

          People

          • Assignee:
            Sean Owen
            Reporter:
            Sean Owen
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Due:
              Created:
              Updated:
              Resolved:

              Development