Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14849 )
Change subject: KUDU-2992: Avoid sending duplicated requests in catalog_manager ...................................................................... KUDU-2992: Avoid sending duplicated requests in catalog_manager When the master sent a large number of rpc requests to a tserver, and the tserver didn't response immediately, catalog manager may continue to send duplicate requests in ProcessTabletReport(), that would cause rpc timeouts and high load on both masters and tservers. This patch add a tablet_id index to pending_tasks, catalog manager would not send duplicate requests to a tablet with the same alter type when handling a tablet report from a tablet server. If the existing RPC task failed, catalog manager would create it the next time the tablet server heartbeats. This patch also fix HandleResponse() in AsyncDeleteReplica to avoid unnecessary retries. Change-Id: If090723821bd78e14a3c54a35cb5e471320002e9 Reviewed-on: http://gerrit.cloudera.org:8080/14849 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/master/catalog_manager.cc M src/kudu/master/catalog_manager.h M src/kudu/master/master-test.cc M src/kudu/tserver/ts_tablet_manager.cc 4 files changed, 197 insertions(+), 57 deletions(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/14849 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: If090723821bd78e14a3c54a35cb5e471320002e9 Gerrit-Change-Number: 14849 Gerrit-PatchSet: 7 Gerrit-Owner: Yifan Zhang <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yifan Zhang <[email protected]>
