Details

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

      Description

      Hbase smoke artifacts aren't working anymore:

      • dependencies are broken
      • code doesn't compile
      1. BIGTOP-1302.patch
        2 kB
        Konstantin Boudnik
      2. BIGTOP-1302.patch
        10 kB
        Konstantin Boudnik
      3. BIGTOP-1302.patch
        13 kB
        Konstantin Boudnik

        Activity

        Hide
        cos Konstantin Boudnik added a comment -

        Committed to the master.

        Show
        cos Konstantin Boudnik added a comment - Committed to the master.
        Hide
        cos Konstantin Boudnik added a comment -

        Actually, this would be totally great to have, as the test coverage would be wider and we won't need to deal with potential bit-rot. I will open a separate JIRA for it. Thanks for the review.

        Show
        cos Konstantin Boudnik added a comment - Actually, this would be totally great to have, as the test coverage would be wider and we won't need to deal with potential bit-rot. I will open a separate JIRA for it. Thanks for the review.
        Hide
        apurtell Andrew Purtell added a comment - - edited

        Patch looks good.

        In the future I wonder if it would be possible to invoke the tests shipping in the HBase test jar rather than clone the unit test code. See for example how the Phoenix package invokes its end to end test suite.

        Show
        apurtell Andrew Purtell added a comment - - edited Patch looks good. In the future I wonder if it would be possible to invoke the tests shipping in the HBase test jar rather than clone the unit test code. See for example how the Phoenix package invokes its end to end test suite.
        Hide
        cos Konstantin Boudnik added a comment -

        getting there: tests are running now

        Show
        cos Konstantin Boudnik added a comment - getting there: tests are running now
        Hide
        cos Konstantin Boudnik added a comment -

        This version of the patch fixes the artifact creation problem (including broken tests). However, the execution side still needs to be worked on.

        Show
        cos Konstantin Boudnik added a comment - This version of the patch fixes the artifact creation problem (including broken tests). However, the execution side still needs to be worked on.

          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