Uploaded image for project: 'Kudu'
  1. Kudu
  2. KUDU-2962

Fix kudu::itest::FindTabletFollowers() test utility function

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.12.0
    • Component/s: test
    • Labels:

      Description

      The kudu::itest::FindTabletFollowers() function is unsafe: it uses kudu::itest::FindTabletLeader() to generate the result as a complement to tablet servers hosting the leader replica, but it doesn't sanitize the set of tablet servers to make sure it contains only tablet servers hosting replicas of the specified tablet.

      For example, if you have a cluster with 10 tablet servers, and a tablet with 3 tablet replicas, passing the map for all tablet servers in the 10-node cluster would result in FindTabletFollowers() reporting 9 followers. Whoops!

      It's necessary to either fix the implementation of this utility function to sanitize its first argument, or simply get rid of it.

        Attachments

          Activity

            People

            • Assignee:
              bankim Bankim Bhavsar
              Reporter:
              aserbin Alexey Serbin
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: