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

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        16d 21h 22m 1 Sean Owen 04/Jun/11 08:18
        Resolved Resolved Closed Closed
        250d 6h 42m 1 Sean Owen 09/Feb/12 14:00
        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