Sailesh Mukil has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8471 )
Change subject: IMPALA-4671: (part-1) Copy kudu::ServicePool into Impala namespace ...................................................................... IMPALA-4671: (part-1) Copy kudu::ServicePool into Impala namespace *** THIS PATCH IS NOT FOR REVIEW. IT IS JUST UPLOADED TO MAKE THE REVIEW OF PART-2 EASIER IN GERRIT BY SHOWING ONLY THE DIFFS OF THAT PATCH AGAINST THIS PATCH. MORE EXPLANATION BELOW*** The KuduRPC subsystem uses kudu::ServicePool to service all incoming RPCs. Since this lives inside the Kudu codebase, all instrumentation of RPCs flowing through the KuduRPC subsystem is not visible to Impala, as Kudu does not export this instrumentation, but only maintains it internally within kudu::ServicePool. In order to reliably view the instrumentation of all RPCs flowing through KRPC, one option is to modify kudu::ServicePool to take their instrumentation and display it on our webpages, which is very invasive. A second option is to have a parallel implementation of kudu::ServicePool which for all purposes behaves the same way, but instead has this extra code that displays this instrumentation in our webpages. This patch is Part 1 of the second option. In this patch, we simply copy the kudu::ServicePool code into be/src/rpc/. In the next patch (Part 2), we rename it to ImpalaServicePool, and we do the instrumentation such that it shows up on the webpage. We split it up this way into 2 parts so that it will be easy to review. Change-Id: I5bb927d4726d4fe418251b9734a4e232810fef69 Reviewed-on: http://gerrit.cloudera.org:8080/8471 Reviewed-by: Sailesh Mukil <sail...@cloudera.com> Tested-by: Sailesh Mukil <sail...@cloudera.com> --- A be/src/rpc/impala-service-pool.cc A be/src/rpc/impala-service-pool.h 2 files changed, 317 insertions(+), 0 deletions(-) Approvals: Sailesh Mukil: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/8471 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I5bb927d4726d4fe418251b9734a4e232810fef69 Gerrit-Change-Number: 8471 Gerrit-PatchSet: 8 Gerrit-Owner: Sailesh Mukil <sail...@cloudera.com> Gerrit-Reviewer: Lars Volker <l...@cloudera.com> Gerrit-Reviewer: Michael Ho <k...@cloudera.com> Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com>
