Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12585 )
Change subject: Add GetTableLocations tracing ...................................................................... Add GetTableLocations tracing This adds minimal tracing to GetTableLocations. It's useful because adding a trace event for the call makes it easier to count the calls in bulk in a trace, and having the requestor string traced makes it possible to attribute GetTableLocations calls to specific clients in many cases, given that the trace will also include the hostport of the client. Change-Id: I92e0dd18de52596f2b9a419c1b6cf7a5666d24bc Reviewed-on: http://gerrit.cloudera.org:8080/12585 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/master/catalog_manager.cc M src/kudu/master/master_service.cc 2 files changed, 5 insertions(+), 1 deletion(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/12585 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I92e0dd18de52596f2b9a419c1b6cf7a5666d24bc Gerrit-Change-Number: 12585 Gerrit-PatchSet: 3 Gerrit-Owner: Will Berkeley <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Will Berkeley <[email protected]>
