Details

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

      Description

      More sophistication with SLive, where we can start testing variational load of the cluster, etc.

      1. BIGTOP-1209.patch
        4 kB
        Virginia Wang
      2. BIGTOP-1209.patch
        6 kB
        Virginia Wang
      3. BIGTOP-1209.patch
        7 kB
        Virginia Wang

        Activity

        Hide
        Konstantin Boudnik added a comment -

        Pushed to the master. Thanks Virginia!

        Show
        Konstantin Boudnik added a comment - Pushed to the master. Thanks Virginia!
        Hide
        Konstantin Boudnik added a comment -

        +1 on the patch. Will commit it shortly.

        Show
        Konstantin Boudnik added a comment - +1 on the patch. Will commit it shortly.
        Hide
        Virginia Wang added a comment -

        Update patches with the fixes mentioned above.

        Show
        Virginia Wang added a comment - Update patches with the fixes mentioned above.
        Hide
        Konstantin Boudnik added a comment -
        • License boiler plate should have the empty line, but they shouldn't have <p> in it. So, it should look like
           * with the License.  You may obtain a copy of the License at
           * 
           * http://www.apache.org/licenses/LICENSE-2.0
           * 
          

          as can be seen in here

        • the class doc says /** SLive Stress test which isn't fully correct, as you're contributing longevity test that can be used for stress and other purposes. Please change this.
        • {{ public void cleanup() { }} can be made private
        • asserts need to have meaningful messages

        The rest looks good.

        Show
        Konstantin Boudnik added a comment - License boiler plate should have the empty line, but they shouldn't have <p> in it. So, it should look like * with the License. You may obtain a copy of the License at * * http://www.apache.org/licenses/LICENSE-2.0 * as can be seen in here the class doc says /** SLive Stress test which isn't fully correct, as you're contributing longevity test that can be used for stress and other purposes. Please change this. {{ public void cleanup() { }} can be made private asserts need to have meaningful messages The rest looks good.
        Hide
        Virginia Wang added a comment -

        New patch added with write/append/read sizes altered to shorten the testing time. Also added iteration function for the test and the fix Cos mentioned.

        Show
        Virginia Wang added a comment - New patch added with write/append/read sizes altered to shorten the testing time. Also added iteration function for the test and the fix Cos mentioned.
        Hide
        Konstantin Boudnik added a comment -

        Great! In the next iteration please get rid of + * <p/> in the license boiler plate.
        Another idea that you've expressed off-line is to have a way to loop the execution. I think it worth adding here as well!

        Show
        Konstantin Boudnik added a comment - Great! In the next iteration please get rid of + * <p/> in the license boiler plate. Another idea that you've expressed off-line is to have a way to loop the execution. I think it worth adding here as well!
        Hide
        Virginia Wang added a comment -

        Added patch for SLive. Currently using 100 files as default. Will be changing the default append/write/read size to control default test time.

        Show
        Virginia Wang added a comment - Added patch for SLive. Currently using 100 files as default. Will be changing the default append/write/read size to control default test time.

          People

          • Assignee:
            Virginia Wang
            Reporter:
            Konstantin Boudnik
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development