Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.0.0
    • Fix Version/s: 1.1.0
    • Component/s: general
    • Labels:
      None

      Description

      Bump up Mahout version to 0.11.0, Expecting enhancement on Spark support

        Issue Links

          Activity

          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user youngwookim opened a pull request:

          https://github.com/apache/bigtop/pull/27

          BIGTOP-1963. Upgrade Mahout to 0.11.0

          You can merge this pull request into a Git repository by running:

          $ git pull https://github.com/youngwookim/bigtop BIGTOP-1963

          Alternatively you can review and apply these changes as the patch at:

          https://github.com/apache/bigtop/pull/27.patch

          To close this pull request, make a commit to your master/trunk branch
          with (at least) the following in the commit message:

          This closes #27


          commit 583cf2ebc8110159d62f638ca0c76d60db4016c8
          Author: YoungWoo Kim <ywkim@apache.org>
          Date: 2015-06-01T07:49:50Z

          BIGTOP-1963. Upgrade Mahout to 0.11.0


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user youngwookim opened a pull request: https://github.com/apache/bigtop/pull/27 BIGTOP-1963 . Upgrade Mahout to 0.11.0 You can merge this pull request into a Git repository by running: $ git pull https://github.com/youngwookim/bigtop BIGTOP-1963 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/bigtop/pull/27.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #27 commit 583cf2ebc8110159d62f638ca0c76d60db4016c8 Author: YoungWoo Kim <ywkim@apache.org> Date: 2015-06-01T07:49:50Z BIGTOP-1963 . Upgrade Mahout to 0.11.0
          Hide
          warwithin YoungWoo Kim added a comment -

          Send a PR to bump up Mahout version to 0.11.0

          Verified by running existing Mahout smoke tests and Mahout's spark-shell manually on VM.

          Show
          warwithin YoungWoo Kim added a comment - Send a PR to bump up Mahout version to 0.11.0 Verified by running existing Mahout smoke tests and Mahout's spark-shell manually on VM.
          Hide
          cos Konstantin Boudnik added a comment -

          Looks good (without actually testing it, so take with a grain of salt)
          +1

          Show
          cos Konstantin Boudnik added a comment - Looks good (without actually testing it, so take with a grain of salt) +1
          Hide
          warwithin YoungWoo Kim added a comment -

          Thank you Konstantin Boudnik for reviewing the patch Once BIGTOP-1953 is committed, I'll commit it.

          Show
          warwithin YoungWoo Kim added a comment - Thank you Konstantin Boudnik for reviewing the patch Once BIGTOP-1953 is committed, I'll commit it.
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user jayunit100 commented on the pull request:

          https://github.com/apache/bigtop/pull/27#issuecomment-132414236

          Thanks have we tested the new mahout with spark?

          Show
          githubbot ASF GitHub Bot added a comment - Github user jayunit100 commented on the pull request: https://github.com/apache/bigtop/pull/27#issuecomment-132414236 Thanks have we tested the new mahout with spark?
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user youngwookim commented on the pull request:

          https://github.com/apache/bigtop/pull/27#issuecomment-132418676

          @jayunit100 I did it manually using Mahout's spark-shell. At this point, there is no integration test for Mahout-Spark at Bigtop smokes.

          Show
          githubbot ASF GitHub Bot added a comment - Github user youngwookim commented on the pull request: https://github.com/apache/bigtop/pull/27#issuecomment-132418676 @jayunit100 I did it manually using Mahout's spark-shell. At this point, there is no integration test for Mahout-Spark at Bigtop smokes.
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user jayunit100 commented on the pull request:

          https://github.com/apache/bigtop/pull/27#issuecomment-132576897

          Great.
          So we can announce that with this patch bigtop now supports mahout-spark?

          Show
          githubbot ASF GitHub Bot added a comment - Github user jayunit100 commented on the pull request: https://github.com/apache/bigtop/pull/27#issuecomment-132576897 Great. So we can announce that with this patch bigtop now supports mahout-spark?
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user smarthi commented on the pull request:

          https://github.com/apache/bigtop/pull/27#issuecomment-132595515

          This is great news @jayunit100

          Show
          githubbot ASF GitHub Bot added a comment - Github user smarthi commented on the pull request: https://github.com/apache/bigtop/pull/27#issuecomment-132595515 This is great news @jayunit100
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user youngwookim commented on the pull request:

          https://github.com/apache/bigtop/pull/27#issuecomment-132619941

          Yup. If users have appropriate system variables, Mahout's spark shell should work. I followed instructions from https://mahout.apache.org/users/sparkbindings/play-with-shell.html

          Show
          githubbot ASF GitHub Bot added a comment - Github user youngwookim commented on the pull request: https://github.com/apache/bigtop/pull/27#issuecomment-132619941 Yup. If users have appropriate system variables, Mahout's spark shell should work. I followed instructions from https://mahout.apache.org/users/sparkbindings/play-with-shell.html
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user jayunit100 commented on the pull request:

          https://github.com/apache/bigtop/pull/27#issuecomment-132655769

          +1 let's merge this !!!

          Show
          githubbot ASF GitHub Bot added a comment - Github user jayunit100 commented on the pull request: https://github.com/apache/bigtop/pull/27#issuecomment-132655769 +1 let's merge this !!!
          Hide
          warwithin YoungWoo Kim added a comment -

          Thanks Jay! Will commit shortly.

          Show
          warwithin YoungWoo Kim added a comment - Thanks Jay! Will commit shortly.

            People

            • Assignee:
              warwithin YoungWoo Kim
              Reporter:
              warwithin YoungWoo Kim
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development