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

Use a random ZK client port in unit tests so we can run them in parallel

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 0.94.0
    • None
    • None
    • Reviewed
    • HTable ctors which only take table name are removed in 0.94

    Description

      The hard-coded ZK client port has long been a problem for running HBase test suite in parallel. The mini ZK cluster should run on a random free port, and that port should be passed to all parts of the unit tests that need to talk to the mini cluster. In fact, randomizing the port exposes a lot of places in the code where a new configuration is instantiated, and as a result the client tries to talk to the default ZK client port and times out.

      The initial fix is for 0.89-fb, where it already allows to run unit tests in parallel in 10 minutes. A fix for the trunk will follow.

      Attachments

        1. D279.92
          4 kB
          Ted Yu
        2. D279-trunk-v7.txt
          50 kB
          Ted Yu
        3. ASF.LICENSE.NOT.GRANTED--D279.7.patch
          50 kB
          Phabricator
        4. ASF.LICENSE.NOT.GRANTED--D279.6.patch
          89 kB
          Phabricator
        5. D279-trunk-v5.txt
          49 kB
          Ted Yu
        6. ASF.LICENSE.NOT.GRANTED--D279.5.patch
          50 kB
          Phabricator
        7. ASF.LICENSE.NOT.GRANTED--D279.4.patch
          50 kB
          Phabricator
        8. ASF.LICENSE.NOT.GRANTED--D255.2.patch
          51 kB
          Phabricator
        9. ASF.LICENSE.NOT.GRANTED--D279.3.patch
          39 kB
          Phabricator
        10. ASF.LICENSE.NOT.GRANTED--D279.2.patch
          38 kB
          Phabricator
        11. 4746-trunk-v2.txt
          33 kB
          Ted Yu
        12. ASF.LICENSE.NOT.GRANTED--D279.1.patch
          31 kB
          Phabricator
        13. ASF.LICENSE.NOT.GRANTED--D255.1.patch
          49 kB
          Phabricator

        Activity

          People

            mikhail Mikhail Gryzykhin
            mikhail Mikhail Gryzykhin
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: