Uploaded image for project: 'Samza'
  1. Samza
  2. SAMZA-160

Remove Scala 2.8 support, default to 2.10

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.7.0
    • Component/s: build
    • Labels:
      None

      Description

      Per the discussion on the list, 2.8 has been voted off the island. Long live 2.10.

      1. SAMZA-160-3.patch
        4 kB
        Jakob Homan
      2. SAMZA-160-2.patch
        3 kB
        Jakob Homan
      3. SAMZA-160.patch
        0.8 kB
        Jakob Homan

        Issue Links

          Activity

          Hide
          jghoman Jakob Homan added a comment -

          Does the needful.

          Show
          jghoman Jakob Homan added a comment - Does the needful.
          Hide
          jghoman Jakob Homan added a comment -

          Simple patch. No rb.

          Show
          jghoman Jakob Homan added a comment - Simple patch. No rb.
          Hide
          criccomini Chris Riccomini added a comment -

          1. Can you update the docs as well?

          docs/learn/tutorials/0.7.0/remote-debugging-samza.md:./gradlew -PscalaVersion=2.8.1 clean publishToMavenLocal
          README.md:Samza builds with [Scala](http://www.scala-lang.org/) 2.9.2 and [YARN](http://hadoop.apache.org/docs/current/hadoop-yarn/hadoop-yarn-site/YARN.html) 2.2.0, by default. Use the -PscalaVersion switches to change Scala versions. Samza supports building Scala with 2.8.1, 2.9.2, or 2.10.
          README.md:    ./gradlew -PscalaVersion=2.8.1 clean build
          RELEASE.md:    ./gradlew -PscalaVersion=2.8.1 clean publishToMavenLocal
          

          2. Can you open a second JIRA to clean up CachedStore so we don't have to do all the Scala 2.8 back bending in it (or fix it in this JIRA)?

          3. I'll trade you a review for SAMZA-158.

          Show
          criccomini Chris Riccomini added a comment - 1. Can you update the docs as well? docs/learn/tutorials/0.7.0/remote-debugging-samza.md:./gradlew -PscalaVersion=2.8.1 clean publishToMavenLocal README.md:Samza builds with [Scala](http://www.scala-lang.org/) 2.9.2 and [YARN](http://hadoop.apache.org/docs/current/hadoop-yarn/hadoop-yarn-site/YARN.html) 2.2.0, by default. Use the -PscalaVersion switches to change Scala versions. Samza supports building Scala with 2.8.1, 2.9.2, or 2.10. README.md: ./gradlew -PscalaVersion=2.8.1 clean build RELEASE.md: ./gradlew -PscalaVersion=2.8.1 clean publishToMavenLocal 2. Can you open a second JIRA to clean up CachedStore so we don't have to do all the Scala 2.8 back bending in it (or fix it in this JIRA)? 3. I'll trade you a review for SAMZA-158 .
          Hide
          jghoman Jakob Homan added a comment -

          Updated patch with doc fix. Will open new jira for cached store.

          Show
          jghoman Jakob Homan added a comment - Updated patch with doc fix. Will open new jira for cached store.
          Hide
          criccomini Chris Riccomini added a comment -

          I noticed two other things:

          1. Delete samza-kafka/lib/kafka_2.8.1*
          2. hello-samza needs to be updated to use 2.10 instead of 2.8.1 (bin/grid, at least)

          You can do (2) after this commit is merged, but (1) I think should be done now.

          Show
          criccomini Chris Riccomini added a comment - I noticed two other things: 1. Delete samza-kafka/lib/kafka_2.8.1* 2. hello-samza needs to be updated to use 2.10 instead of 2.8.1 (bin/grid, at least) You can do (2) after this commit is merged, but (1) I think should be done now.
          Hide
          jghoman Jakob Homan added a comment -

          Jars go bye bye.

          Show
          jghoman Jakob Homan added a comment - Jars go bye bye.
          Hide
          criccomini Chris Riccomini added a comment -

          +1

          Show
          criccomini Chris Riccomini added a comment - +1
          Hide
          jghoman Jakob Homan added a comment -

          Committed. Resolving.

          Show
          jghoman Jakob Homan added a comment - Committed. Resolving.

            People

            • Assignee:
              jghoman Jakob Homan
              Reporter:
              jghoman Jakob Homan
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development