Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-4703

Improvements in tests

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 0.92.0
    • Fix Version/s: 0.92.0
    • Component/s: test
    • Labels:
      None
    • Environment:

      all

    • Hadoop Flags:
      Reviewed

      Description

      Global:

      • when possible, make the test using the default cluster configuration for the number of region (1 instead of 2 or 3). This allows a faster stop/start, and is a step toward a shared cluster configuration.
      • 'sleep': lower or remove the sleep based synchronisation in the tests (in HBaseTestingUtility, TestGlobalMemStoreSize, TestAdmin, TestCoprocessorEndpoint, TestHFileOutputFormat, TestLruBlockCache, TestServerCustomProtocol, TestReplicationSink)
      • Optimize 'put' by setting setWriteToWAL to false, when the 'put' is big or in a loop. Not done for tests that rely on the WAL.

      Local issues:

      • TestTableInputFormatScan fully deletes the hadoop.tmp.dir directory on tearDown, that makes it impossible to use in // with another test using this directory
      • TestIdLock logs too much (9000 lines per seconds). Test time lowered to 15 seconds to make it a part of the small subset
      • TestMemoryBoundedLogMessageBuffer useless System.out.println
      • io.hfile.TestReseekTo useless System.out.println
      • TestTableInputFormat does not shutdown the cluster
      • testGlobalMemStore does not shutdown the cluster
      • rest.client.TestRemoteAdmin: simplified, does not use local admin, single test instead of two.
      • HBaseTestingUtility#ensureSomeRegionServersAvailable starts only one server, should start the number of missing server instead.
      • TestMergeTool should starts/stops the dfs cluster with HBaseTestingUtility

        Attachments

        1. 20111030_4703_all.patch
          68 kB
          Nicolas Liochon
        2. 20111030_4703_all.v2.patch
          68 kB
          Nicolas Liochon
        3. 20111030_4703_all.v3.patch
          66 kB
          Nicolas Liochon
        4. 20111030_4703_all.v4.patch
          65 kB
          Nicolas Liochon

          Issue Links

            Activity

              People

              • Assignee:
                nkeywal Nicolas Liochon
                Reporter:
                nkeywal Nicolas Liochon
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: