Yifan Zhang has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19594 )
Change subject: [KUDU-3452] Make validate tablet creating task not affected ...................................................................... [KUDU-3452] Make validate tablet creating task not affected Currently, creating a table with RF=n when the number of healthy tservers is less than n will get stuck. Because catalog manager creates tablets for it will fail and retry continuously. At the same time, creating a table with RF=m also will get stuck even if there are more than m healthy tservers. Because catalog manager will return when finds a tablet-creating task failed and will not try to select replicas for other PREPARING tablets. For example, creating a three replicas table times out when one of three tablet servers becomes unavailable. After that, creating a two-replicas table also will timeout even if there are enough tablet servers to place its replicas. The validate two-replicas table-creating task will be affected by invalidate three-replicas table-creating task. This patch fixes this problem. If a task of creating tablet fail, it will not return immediately, but let other tasks of creating other tablets keep on running. Change-Id: I64668651d0e8f58b92cfb841bdb20617de6776f9 Reviewed-on: http://gerrit.cloudera.org:8080/19594 Reviewed-by: Yifan Zhang <[email protected]> Tested-by: Yifan Zhang <[email protected]> --- M src/kudu/integration-tests/create-table-itest.cc M src/kudu/master/catalog_manager.cc 2 files changed, 109 insertions(+), 5 deletions(-) Approvals: Yifan Zhang: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/19594 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I64668651d0e8f58b92cfb841bdb20617de6776f9 Gerrit-Change-Number: 19594 Gerrit-PatchSet: 25 Gerrit-Owner: Wang Xixu <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: KeDeng <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Wang Xixu <[email protected]> Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Reviewer: Yuqi Du <[email protected]>
