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

25s sleep when expiring sessions in tests

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 0.90.4
    • 0.94.0
    • None
    • None

    Description

      There's a hardcoded 25 seconds sleep in HBaseTestingUtility.expireSession:

      int sessionTimeout = 5 * 1000; // 5 seconds
      ...
      final long sleep = sessionTimeout * 5L;
      LOG.info("ZK Closed Session 0x" + Long.toHexString(sessionID) +
        "; sleeping=" + sleep);
      Thread.sleep(sleep);
      

      I'm pretty sure this can be lowered at lot, and it would speed up a couple of tests. The only thing I'm afraid of is if this was made to accomodate flaky tests.

      Attachments

        Issue Links

          Activity

            People

              nkeywal Nicolas Liochon
              jdcryans Jean-Daniel Cryans
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: