Hadoop Common
  1. Hadoop Common
  2. HADOOP-8257

Auto-HA: TestZKFailoverControllerStress occasionally fails with Mockito error

    Details

    • Hadoop Flags:
      Reviewed

      Description

      Once in a while I've seen the following in TestZKFailoverControllerStress:

      Unfinished stubbing detected here: -> at org.apache.hadoop.ha.TestZKFailoverControllerStress.testRandomHealthAndDisconnects(TestZKFailoverControllerStress.java:118) E.g. thenReturn() may be missing....

      This is because we set up the mock answers after starting the ZKFCs. So if the ZKFC calls the mock object while it's in the middle of the setup, this exception occurs.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        35m 57s 1 Todd Lipcon 06/Apr/12 22:20
        Todd Lipcon made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Hadoop Flags Reviewed [ 10343 ]
        Fix Version/s Auto Failover (HDFS-3042) [ 12320350 ]
        Resolution Fixed [ 1 ]
        Hide
        Todd Lipcon added a comment -

        Jenkins won't run on this since it's on a branch. I verified by changing the test runtime to 3 seconds and looping it. Without the patch, it failed with the mockito error after 3 or 4 minutes. I then looped with the patch for 15 minutes without a failure.

        Show
        Todd Lipcon added a comment - Jenkins won't run on this since it's on a branch. I verified by changing the test runtime to 3 seconds and looping it. Without the patch, it failed with the mockito error after 3 or 4 minutes. I then looped with the patch for 15 minutes without a failure.
        Hide
        Eli Collins added a comment -

        +1 pending jenkins, good catch

        Show
        Eli Collins added a comment - +1 pending jenkins, good catch
        Todd Lipcon made changes -
        Field Original Value New Value
        Attachment hadoop-8257.txt [ 12521764 ]
        Hide
        Todd Lipcon added a comment -

        This patch to the test should fix the issue.

        Show
        Todd Lipcon added a comment - This patch to the test should fix the issue.
        Todd Lipcon created issue -

          People

          • Assignee:
            Todd Lipcon
            Reporter:
            Todd Lipcon
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development