Details

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

      Description

      Like a HCFS, HDFS, let's also have YARN smoke tests to have complete coverage of Hadoop integration.

        Activity

        Hide
        cos Konstantin Boudnik added a comment -

        A patch easier than I expected

        Show
        cos Konstantin Boudnik added a comment - A patch easier than I expected
        Hide
        cos Konstantin Boudnik added a comment -

        I ran old-way tests as well as new smoking way: all works fine

        Show
        cos Konstantin Boudnik added a comment - I ran old-way tests as well as new smoking way: all works fine
        Hide
        rvs Roman Shaposhnik added a comment -

        Why is this needed?

        systemProperty 'hcfs.root.username', System.properties['user.name']
        
        Show
        rvs Roman Shaposhnik added a comment - Why is this needed? systemProperty 'hcfs.root.username', System.properties['user.name']
        Hide
        cos Konstantin Boudnik added a comment -

        Ah, good catch - this is the copy-cut from hdfs suite. Will remove it in a sec.

        Show
        cos Konstantin Boudnik added a comment - Ah, good catch - this is the copy-cut from hdfs suite. Will remove it in a sec.
        Hide
        cos Konstantin Boudnik added a comment -

        Good catch Roman Shaposhnik. It wasn't bad, but definitely useless Here's new patch.

        Show
        cos Konstantin Boudnik added a comment - Good catch Roman Shaposhnik . It wasn't bad, but definitely useless Here's new patch.
        Hide
        rvs Roman Shaposhnik added a comment -

        +1. LGTM.

        Show
        rvs Roman Shaposhnik added a comment - +1. LGTM.
        Hide
        cos Konstantin Boudnik added a comment -

        and committed/pushed to the master now.

        Show
        cos Konstantin Boudnik added a comment - and committed/pushed to the master now.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development