You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@kudu.apache.org by "Bankim Bhavsar (Jira)" <ji...@apache.org> on 2019/10/23 00:43:00 UTC

[jira] [Updated] (KUDU-2962) Fix kudu::itest::FindTabletFollowers() test utility function

     [ https://issues.apache.org/jira/browse/KUDU-2962?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bankim Bhavsar updated KUDU-2962:
---------------------------------
    Status: In Review  (was: Open)

> Fix kudu::itest::FindTabletFollowers() test utility function
> ------------------------------------------------------------
>
>                 Key: KUDU-2962
>                 URL: https://issues.apache.org/jira/browse/KUDU-2962
>             Project: Kudu
>          Issue Type: Improvement
>          Components: test
>            Reporter: Alexey Serbin
>            Assignee: Bankim Bhavsar
>            Priority: Minor
>              Labels: newbie
>
> 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.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)