Details

    • Sub-task
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 2.0.0-beta-2, 2.0.0
    • None
    • None
    • Reviewed

    Description

      Timedout in HBASE-19906

      Comparing a local run (16seconds total) to a timed out run up on jenkins, I see it takes my local test 5 seconds to get the STOPPED server log line. On jenkins in this timed out test it takes 30 seconds. Test is still running when it is killed. Let me make it a medium test.

      https://builds.apache.org/view/H-L/view/HBase/job/PreCommit-HBASE-Build/lastCompletedBuild/testReport/org.apache.hadoop.hbase.coprocessor/TestCoprocessorShortCircuitRPC/org_apache_hadoop_hbase_coprocessor_TestCoprocessorShortCircuitRPC/

      Attachments

        1. HBASE-19908.master.001.patch
          2 kB
          Michael Stack

        Issue Links

          Activity

            stack Michael Stack added a comment - - edited

            .001 is what I pushed to master and branch-2. Leaving open to see if this fixes it. It makes the test medium rather than small

            stack Michael Stack added a comment - - edited .001 is what I pushed to master and branch-2. Leaving open to see if this fixes it. It makes the test medium rather than small

            Let's use HBASE-19878 directly for making these minor changes?
            Otherwise there's all this process of linking everything together - new jiras, HBASE-19878 and HBASE-19147
            fyi Apache9.

            appy Apekshit Sharma added a comment - Let's use HBASE-19878 directly for making these minor changes? Otherwise there's all this process of linking everything together - new jiras, HBASE-19878 and HBASE-19147 fyi Apache9 .
            zhangduo Duo Zhang added a comment -

            The problem is that we can not find them at once...

            zhangduo Duo Zhang added a comment - The problem is that we can not find them at once...
            stack Michael Stack added a comment -

            And unusual that there is a pattern as in these last three... Will try better to group them.

            stack Michael Stack added a comment - And unusual that there is a pattern as in these last three... Will try better to group them.
            zhangduo Duo Zhang added a comment -

            Maybe we can increase the time limit a bit? For example, to 1 minute? I see that some tests which do not start a mini cluster could also timeout...

            zhangduo Duo Zhang added a comment - Maybe we can increase the time limit a bit? For example, to 1 minute? I see that some tests which do not start a mini cluster could also timeout...
            stack Michael Stack added a comment -

            Maybe we can increase the time limit a bit?

            Refguide has this:

            http://hbase.apache.org/book.html#hbase.unittests

            15s/50s/Everything else...

            Funny. Says small tests should not execute minicluster.

            Then HBaseClassTestRule has

            for (Class<?> c : categories[0].value()) {
            if (c == SmallTests.class)

            { // See SmallTests. Supposed to run 15 seconds. return 30; }

            else if (c == MediumTests.class)

            { // See MediumTests. Supposed to run 50 seconds. return 180; }

            else if (c == LargeTests.class) {
            // Let large tests have a ten minute timeout.
            return TimeUnit.MINUTES.toSeconds(10);

            .... this is time for each individual method in the test suite (as I read it... not for whole class).

            So, we've already gone up from the 15 seconds we talk of in the refguide?

            Or, just up small tests to 60seconds? Leave the rest (i like the ten minutes total).

            Apache9

            stack Michael Stack added a comment - Maybe we can increase the time limit a bit? Refguide has this: http://hbase.apache.org/book.html#hbase.unittests 15s/50s/Everything else... Funny. Says small tests should not execute minicluster. Then HBaseClassTestRule has for (Class<?> c : categories [0] .value()) { if (c == SmallTests.class) { // See SmallTests. Supposed to run 15 seconds. return 30; } else if (c == MediumTests.class) { // See MediumTests. Supposed to run 50 seconds. return 180; } else if (c == LargeTests.class) { // Let large tests have a ten minute timeout. return TimeUnit.MINUTES.toSeconds(10); .... this is time for each individual method in the test suite (as I read it... not for whole class). So, we've already gone up from the 15 seconds we talk of in the refguide? Or, just up small tests to 60seconds? Leave the rest (i like the ten minutes total). Apache9
            zhangduo Duo Zhang added a comment -

            Yes, only up small to 60s without changing others.

            zhangduo Duo Zhang added a comment - Yes, only up small to 60s without changing others.
            stack Michael Stack added a comment -

            Yes, only up small to 60s without changing others.

            I'll do it on the next small test timeout....

            stack Michael Stack added a comment - Yes, only up small to 60s without changing others. I'll do it on the next small test timeout....
            hudson Hudson added a comment -

            FAILURE: Integrated in Jenkins build HBase-Trunk_matrix #4506 (See https://builds.apache.org/job/HBase-Trunk_matrix/4506/)
            HBASE-19908 TestCoprocessorShortCircuitRPC Timeout.... (stack: rev bc0e9ce8f462d8b3f4ca67ad80d455edb39605bf)

            • (edit) hbase-server/src/test/java/org/apache/hadoop/hbase/coprocessor/TestCoprocessorShortCircuitRPC.java
            hudson Hudson added a comment - FAILURE: Integrated in Jenkins build HBase-Trunk_matrix #4506 (See https://builds.apache.org/job/HBase-Trunk_matrix/4506/ ) HBASE-19908 TestCoprocessorShortCircuitRPC Timeout.... (stack: rev bc0e9ce8f462d8b3f4ca67ad80d455edb39605bf) (edit) hbase-server/src/test/java/org/apache/hadoop/hbase/coprocessor/TestCoprocessorShortCircuitRPC.java
            stack Michael Stack added a comment -

            Apache9 Upped timeout as addendum on HBASE-19911.

            stack Michael Stack added a comment - Apache9 Upped timeout as addendum on HBASE-19911 .
            zhangduo Duo Zhang added a comment -

            Resolve.

            zhangduo Duo Zhang added a comment - Resolve.

            People

              stack Michael Stack
              stack Michael Stack
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: