Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/9384 )
Change subject: IMPALA-6347: Monitor queue depth size for outgoing RPCs for Reactor threads ...................................................................... Patch Set 3: (4 comments) http://gerrit.cloudera.org:8080/#/c/9384/2/be/src/rpc/rpc-mgr.cc File be/src/rpc/rpc-mgr.cc: http://gerrit.cloudera.org:8080/#/c/9384/2/be/src/rpc/rpc-mgr.cc@203 PS2, Line 203: in_flight().size(); > Why are we counting the inbound connections? I thought this change was abou No, that was a honest mistake. Moved it to count outbound calls. http://gerrit.cloudera.org:8080/#/c/9384/2/be/src/rpc/rpc-mgr.cc@208 PS2, Line 208: Value per_conn_metrics(kArrayType); > Should we add a check here that the remote IPs are actually unique? That co In what case would it not be unique? I think we can rely on the fact that KRPC gives us a guarantee that it will only return the active connections that it has, which have to be unique. Plus checking if they're unique would require us to keep a hash map and keep comparing, which seems like a lot of extra effort just to be extra careful. http://gerrit.cloudera.org:8080/#/c/9384/2/www/rpcz.tmpl File www/rpcz.tmpl: http://gerrit.cloudera.org:8080/#/c/9384/2/www/rpcz.tmpl@98 PS2, Line 98: : {{?servers}} : <h2>Impala RPC Services : <button class="btn btn-warning btn-xs" onClick="reset_all();"> : Reset all : </button> > I think the <tbody> should be empty when using DataTables (see www/query_fi Done http://gerrit.cloudera.org:8080/#/c/9384/2/www/rpcz.tmpl@237 PS2, Line 237: // Add more fields to the keys here when more metrics become available. > I think this needs to be change now to update the table through the DataTab This took a while to figure out how I would pass the JSON as a data source to the DataTable, but I was able to figure it out. I got rid of this function and added a new one that updates the DataTable itself. -- To view, visit http://gerrit.cloudera.org:8080/9384 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I84fea531e98d3e84fcc57bf7533655218bc91f4c Gerrit-Change-Number: 9384 Gerrit-PatchSet: 3 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> Gerrit-Comment-Date: Thu, 22 Feb 2018 20:24:09 +0000 Gerrit-HasComments: Yes