Details

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

      Description

      It appears we will need to update to gradle 2.0 for BIGTOP-1222 to work properly due to some changes in gradle 2x subprojects. in particular the groovy plugin seems modified significantly. http://www.gradle.org/docs/2.0/release-notes#changes-to-groovy-and-scala-plugins

      1. BIGTOP-1404.0.RETRO.patch
        2 kB
        jay vyas
      2. BIGTOP-1402.patch
        2 kB
        jay vyas

        Issue Links

          Activity

          Hide
          jayunit100 jay vyas added a comment -

          we are pretty far behind on gradle (1x). time to upgrade. i assume this patch updates the whole toolchain properly. but i dont know how it works really that well so maybe i made a mistake.

          Im actually not even sure why gradle is in the toolchain? It doesnt seem required for running hadoop. maybe we are using the toolchain for more than that though.

          Show
          jayunit100 jay vyas added a comment - we are pretty far behind on gradle (1x). time to upgrade. i assume this patch updates the whole toolchain properly. but i dont know how it works really that well so maybe i made a mistake. Im actually not even sure why gradle is in the toolchain? It doesnt seem required for running hadoop. maybe we are using the toolchain for more than that though.
          Hide
          jayunit100 jay vyas added a comment -

          tl;dr (patch submitted ^^^) let me know if it looks good, or if there is a particular way i should test it

          Show
          jayunit100 jay vyas added a comment - tl;dr (patch submitted ^^^) let me know if it looks good, or if there is a particular way i should test it
          Hide
          jayunit100 jay vyas added a comment - - edited

          Konstantin Boudnik or Roman Shaposhnik looks okay to push in (needed for BIGTOP-1222 test overhaul) ? thanks...

          Show
          jayunit100 jay vyas added a comment - - edited Konstantin Boudnik or Roman Shaposhnik looks okay to push in (needed for BIGTOP-1222 test overhaul) ? thanks...
          Hide
          cos Konstantin Boudnik added a comment -

          Looks good! Could you please update the toolchain Readme file to reflect the change?
          +1 with readme addition.

          Thanks!

          Show
          cos Konstantin Boudnik added a comment - Looks good! Could you please update the toolchain Readme file to reflect the change? +1 with readme addition. Thanks!
          Hide
          jayunit100 jay vyas added a comment -

          okay thanks.... added the README update, and commited.
          Now back to BIGTOP-1222 !

          Show
          jayunit100 jay vyas added a comment - okay thanks.... added the README update, and commited. Now back to BIGTOP-1222 !
          Hide
          cos Konstantin Boudnik added a comment -

          jay vyas, is it possible to attach the actual diff to this JIRA? The change is in the source code, but the patch here is different. Thanks!

          Show
          cos Konstantin Boudnik added a comment - jay vyas , is it possible to attach the actual diff to this JIRA? The change is in the source code, but the patch here is different. Thanks!
          Hide
          jayunit100 jay vyas added a comment -

          oops. sorry about this. let me see what the hell i did... I will attach the patch retroactively if i indeed pushed something from my local branch which differed from the patch.

          Show
          jayunit100 jay vyas added a comment - oops. sorry about this. let me see what the hell i did... I will attach the patch retroactively if i indeed pushed something from my local branch which differed from the patch.
          Hide
          jayunit100 jay vyas added a comment -

          Here is the patch that was actually implemented. just for clarity. not sure wether i pushed something different or not .. a breif check looks the same to me

          Show
          jayunit100 jay vyas added a comment - Here is the patch that was actually implemented. just for clarity. not sure wether i pushed something different or not .. a breif check looks the same to me

            People

            • Assignee:
              jayunit100 jay vyas
              Reporter:
              jayunit100 jay vyas
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development