Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/12229 )
Change subject: IMPALA-7731: Add TxRatio counter to instances and query profile ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/12229/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/12229/1//COMMIT_MSG@16 PS1, Line 16: by instances with a scan over the data read by all scans of the > Let's add TotalBytesExchanged. I think separating TotalBytesRead by storage I added more counters and renamed TxRatio. We don't have a way to tell S3 scans apart in the BE currently so I'd prefer to add that (and counters per storage type, we only maintain them for HDFS and HBase anyways) in a separate change. I'll also wait for your feedback before updating all references to TxRatio in the commit message. -- To view, visit http://gerrit.cloudera.org:8080/12229 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ife7ec78fe42558429c1cbe6e5eba79842bffd648 Gerrit-Change-Number: 12229 Gerrit-PatchSet: 1 Gerrit-Owner: Lars Volker <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Lars Volker <[email protected]> Gerrit-Reviewer: Philip Zeyliger <[email protected]> Gerrit-Comment-Date: Thu, 17 Jan 2019 22:40:06 +0000 Gerrit-HasComments: Yes
