Uploaded image for project: 'Bigtop'
  1. Bigtop
  2. BIGTOP-1663

TestHadoopSmoke is failing because of improper set-up

    Details

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

      Description

      The set-up required for this test is already written, but it's commented out. Looks like it was broken by BIGTOP-1222.

      1. BIGTOP-1663.patch
        1 kB
        Dasha Boudnik

        Issue Links

          Activity

          Hide
          jayunit100 jay vyas added a comment -

          not running from jar, so no way, and no need, to unpack resources....

          Show
          jayunit100 jay vyas added a comment - not running from jar, so no way, and no need, to unpack resources....
          Hide
          cos Konstantin Boudnik added a comment -

          why was it failing in the first place? Couldn't find the jar file to unpack or something?

          Show
          cos Konstantin Boudnik added a comment - why was it failing in the first place? Couldn't find the jar file to unpack or something?
          Hide
          jayunit100 jay vyas added a comment -

          iirc surrounding this guy w/ a try catch block will solve the issue.
          i will submit a patch

          Show
          jayunit100 jay vyas added a comment - iirc surrounding this guy w/ a try catch block will solve the issue. i will submit a patch
          Hide
          jayunit100 jay vyas added a comment -

          hi guys.... ill look and see how it effects smoke-tests .

          Show
          jayunit100 jay vyas added a comment - hi guys.... ill look and see how it effects smoke-tests .
          Hide
          dasha.boudnik Dasha Boudnik added a comment -

          Thanks; committed and pushed.

          Show
          dasha.boudnik Dasha Boudnik added a comment - Thanks; committed and pushed.
          Hide
          cos Konstantin Boudnik added a comment -

          Patch looks good. Please commit.
          jay vyas, I presume it will break some of the new smoke tests, considering that you've made this change earlier. Would you mind look into the fix for it? Thanks!

          Show
          cos Konstantin Boudnik added a comment - Patch looks good. Please commit. jay vyas , I presume it will break some of the new smoke tests, considering that you've made this change earlier. Would you mind look into the fix for it? Thanks!
          Hide
          dasha.boudnik Dasha Boudnik added a comment -

          Patch attached; please review and I'll commit. Thanks!

          Show
          dasha.boudnik Dasha Boudnik added a comment - Patch attached; please review and I'll commit. Thanks!

            People

            • Assignee:
              dasha.boudnik Dasha Boudnik
              Reporter:
              dasha.boudnik Dasha Boudnik
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development