Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/21393 )

Change subject: [tserver] limit number of trace metrics for TabletCopyClient
......................................................................

[tserver] limit number of trace metrics for TabletCopyClient

The trace metrics registry assumes that the number of entries is quite
small, with the current threshold of 100.  In its turn, a thread pool
unconditionally registers and updates its trace metrics while executing
submitted tasks.  Since a tablet server might host thousands of tablet
replicas, it's not a good idea to include the UUID of the tablet into
the name of the download thread pool spawned by every TabletCopyClient
instance.

This is a follow-up to 0d95304fa46ee5d96bcaa934c7660369f2860e06.

Change-Id: I334aa81aaed2378e7cae558bd8bb9e0f0c970fec
Reviewed-on: http://gerrit.cloudera.org:8080/21393
Tested-by: Marton Greber <[email protected]>
Reviewed-by: Marton Greber <[email protected]>
---
M src/kudu/tserver/tablet_copy_client.cc
1 file changed, 10 insertions(+), 2 deletions(-)

Approvals:
  Marton Greber: Looks good to me, approved; Verified

--
To view, visit http://gerrit.cloudera.org:8080/21393
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I334aa81aaed2378e7cae558bd8bb9e0f0c970fec
Gerrit-Change-Number: 21393
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>

Reply via email to