Sailesh Mukil has posted comments on this change. ( )

Change subject: IMPALA-6269: Expose KRPC metrics on debug webpage

Patch Set 14: Code-Review+1

File be/src/rpc/
PS8, Line 76: }
            : ImpalaServicePool::~ImpalaServic
Maybe this is something to consider moving into RpcMethodInfo as a Kudu patch.

In that case, we can get it like we get the handler latency in L273.

Not for now, but if you agree, we can add a TODO here.
File be/src/rpc/
PS8, Line 128: rviceIf* service_p
This might stop us from working with different types of ServiceIfs in the 
future if we need to. But since we don't have any other types of ServiceIfs, 
it's okay for now.

Something to keep a note of though.
File tests/webserver/
PS14, Line 251: 'impala.DataStreamService'
Why not keep this in a local array? Then we can add more services to this test 
as we add them to the code base.

Then we can remove all references to 'DataStreamService' and use the array 
File www/rpcz.tmpl:
PS8, Line 42:         <tr>
            :           <th>Queue Size</th>
            :           <th>Idle threads</th>
            :           <th>Maximum Queue Size</th>
            :           <th>RPCs rejected due to queue overflow</th>
            :         </tr>
            :         <tr>
            :           <td id="{{service_name}}_queue_size">{{queue_size}}</td>
            :           <td 
            :           <td 
            :           </td>
            :           <td 
            :         </tr>
Would specifying a colspan for these guys align them better?

To view, visit
To unsubscribe, visit

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I7af7c1a84a5be82c979ca4ef1edf35167493be3f
Gerrit-Change-Number: 9292
Gerrit-PatchSet: 14
Gerrit-Owner: Lars Volker <>
Gerrit-Reviewer: Lars Volker <>
Gerrit-Reviewer: Michael Ho <>
Gerrit-Reviewer: Sailesh Mukil <>
Gerrit-Comment-Date: Tue, 20 Feb 2018 00:48:29 +0000
Gerrit-HasComments: Yes

Reply via email to