Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8340 )
Change subject: KUDU-2194: use a common trace metric prefix to avoid overflowing intern map ...................................................................... KUDU-2194: use a common trace metric prefix to avoid overflowing intern map Without a prefix, we end up prefixing trace metric names with the corresponding thread pool's name, and each data directory has a uniquely named thread pool. The magic number of data directories needed to overflow the intern map is 34, though it'll be lower in production as the server will have created other thread pools. Change-Id: I2d95e440c3fef76d7cbfe568a014bab9cda7f329 Reviewed-on: http://gerrit.cloudera.org:8080/8340 Reviewed-by: Todd Lipcon <t...@apache.org> Tested-by: Kudu Jenkins Reviewed-by: Andrew Wong <aw...@cloudera.com> --- M src/kudu/fs/data_dirs-test.cc M src/kudu/fs/data_dirs.cc M src/kudu/util/threadpool.cc 3 files changed, 26 insertions(+), 7 deletions(-) Approvals: Todd Lipcon: Looks good to me, approved Kudu Jenkins: Verified Andrew Wong: Looks good to me, but someone else must approve -- To view, visit http://gerrit.cloudera.org:8080/8340 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I2d95e440c3fef76d7cbfe568a014bab9cda7f329 Gerrit-Change-Number: 8340 Gerrit-PatchSet: 2 Gerrit-Owner: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
