Details

    • Type: Sub-task Sub-task
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: QuorumJournalManager (HDFS-3077)
    • Component/s: test
    • Labels:
      None

      Description

      This JIRA is to add a test case which exhaustively tests double-failure scenarios in a 3-node quorum setup. The test instruments the RPCs between the client and the JNs, and injects faults, simulating a dropped RPC. The framework used by this test will also be expanded in future JIRAs for other failure scenarios.

      1. hdfs-3741.txt
        36 kB
        Todd Lipcon

        Activity

        Todd Lipcon made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Hadoop Flags Reviewed [ 10343 ]
        Fix Version/s QuorumJournalManager (HDFS-3077) [ 12322478 ]
        Resolution Fixed [ 1 ]
        Hide
        Todd Lipcon added a comment -

        Committed to branch, thanks.

        Show
        Todd Lipcon added a comment - Committed to branch, thanks.
        Hide
        Aaron T. Myers added a comment -

        This is a pretty baller test, Todd. Good stuff.

        The patch looks good to me, and I agree it makes sense to go ahead and commit it to the branch.

        +1

        Show
        Aaron T. Myers added a comment - This is a pretty baller test, Todd. Good stuff. The patch looks good to me, and I agree it makes sense to go ahead and commit it to the branch. +1
        Todd Lipcon made changes -
        Field Original Value New Value
        Attachment hdfs-3741.txt [ 12538460 ]
        Hide
        Todd Lipcon added a comment -

        Attached patch implements the test as described.

        The test currently fails due to various issues in the code, many of which are pointed out by TODOs.

        That said, I'd like to commit this to the dev branch, as it will provide a reference point and test case for the related fixes. Once we get this test passing cleanly we can move on to adding the next set of injection tests.

        Show
        Todd Lipcon added a comment - Attached patch implements the test as described. The test currently fails due to various issues in the code, many of which are pointed out by TODOs. That said, I'd like to commit this to the dev branch, as it will provide a reference point and test case for the related fixes. Once we get this test passing cleanly we can move on to adding the next set of injection tests.
        Todd Lipcon created issue -

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development