Details

    • Type: New Feature
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.8.0
    • Fix Version/s: 1.0.0
    • Component/s: build, spark
    • Labels:
      None

      Description

      Let the components read Scala version from bigtop.bom. As of now, Kafka and Spark will be affected.

        Issue Links

          Activity

          Hide
          warwithin YoungWoo Kim added a comment -

          Konstantin Boudnik What do you think?

          Show
          warwithin YoungWoo Kim added a comment - Konstantin Boudnik What do you think?
          Hide
          warwithin YoungWoo Kim added a comment -

          BIGTOP-1716.1.patch

          • define SCALA_VERSION in bigtop.bom
          • remove hard-coded SCALA_VERSION from Bigtop
          • specify SCALA_VERSION on Spark build
          Show
          warwithin YoungWoo Kim added a comment - BIGTOP-1716.1.patch define SCALA_VERSION in bigtop.bom remove hard-coded SCALA_VERSION from Bigtop specify SCALA_VERSION on Spark build
          Hide
          cos Konstantin Boudnik added a comment -

          I think it looks good! +1
          I'd perhaps use cut -d'.' -f1,2 instead of grep - a bit more readable IMO - but that's just me

          I will commit it in a bit. A hint - put JIRAs into "Patch available" state when they are ready for review.

          Show
          cos Konstantin Boudnik added a comment - I think it looks good! +1 I'd perhaps use cut -d'.' -f1,2 instead of grep - a bit more readable IMO - but that's just me I will commit it in a bit. A hint - put JIRAs into "Patch available" state when they are ready for review.
          Hide
          cos Konstantin Boudnik added a comment -

          Pushed to the master. Thanks YoungWoo!

          Show
          cos Konstantin Boudnik added a comment - Pushed to the master. Thanks YoungWoo!
          Hide
          warwithin YoungWoo Kim added a comment -

          Konstantin Boudnik Thanks for your comment!

          Show
          warwithin YoungWoo Kim added a comment - Konstantin Boudnik Thanks for your comment!

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development