Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9372 )
Change subject: rowset_metadata: cache min/max encoded keys ...................................................................... Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/9372/6//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/9372/6//COMMIT_MSG@29 PS6, Line 29: Reading from blocks (s): 40.578 38.428 37.093 38.700 This seems to be missing the punch line. Is this implying that, with the patch active, we would cut out this 38sec entirely because we wouldn't read from blocks anymore? -- To view, visit http://gerrit.cloudera.org:8080/9372 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I37d6f7160e3a7188753684e063963110f70e9b8d Gerrit-Change-Number: 9372 Gerrit-PatchSet: 6 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Fri, 30 Mar 2018 02:00:15 +0000 Gerrit-HasComments: Yes
