Uploaded image for project: 'Bigtop'
  1. Bigtop
  2. BIGTOP-2246 Add smoke tests for HCFS
  3. BIGTOP-2245

TestFuseHCFS might fall on '+' op. involving String and GString; class name is wrong

    Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.0.0
    • Fix Version/s: 1.1.0
    • Component/s: tests
    • Labels:
      None

      Description

      Looking into hcfs tests I see that classname doesn't correspond to the file name (not big of a deal, but looks messy). Also, I've noticed that in some cases (like running the test from Groovy smoke env) one of the tests error out with this

       org.codehaus.groovy.runtime.metaclass.MethodSelectionException: Could not find which method plus() to invoke from this list
         public groovy.lang.GString groovy.lang.GString#plus(groovy.lang.GString)
         public groovy.lang.GString groovy.lang.GString#pl org.codehaus.groovy.runtime.metaclass.MethodSelectionException: Could not find which method plus() to invoke from this listus(java.lang.String)
       
         public groovy.lang.GString groovy.lang.GString#plus(groovy.lang.GString)
         public groovy.lang.GString groovy.lang.GString#plus(java.lang.String)
      
      1. BIGTOP-2245.patch
        0.9 kB
        Konstantin Boudnik

        Activity

        Hide
        cos Konstantin Boudnik added a comment -

        While I don't have a clear understanding on the class issue name (jay vyas, could you please chime in as it seems you were the original author of the change, so you might remember what's going on? BTW, there's no such think as TestFuseDFS, so I am not sure how it is relevant).

        Here's the fix for the '+' operator issue. Verified and ready to be committed.

        Show
        cos Konstantin Boudnik added a comment - While I don't have a clear understanding on the class issue name ( jay vyas , could you please chime in as it seems you were the original author of the change, so you might remember what's going on? BTW, there's no such think as TestFuseDFS, so I am not sure how it is relevant). Here's the fix for the '+' operator issue. Verified and ready to be committed.
        Hide
        cos Konstantin Boudnik added a comment -

        Pushed to the master

        Show
        cos Konstantin Boudnik added a comment - Pushed to the master

          People

          • Assignee:
            cos Konstantin Boudnik
            Reporter:
            cos Konstantin Boudnik
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development