Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/9966 )
Change subject: IMPALA-6813: Hedged reads metrics broken when scanning non-HDFS based table ...................................................................... Patch Set 2: > Sailesh: Even though it doesn't count them as hedged reads, does it > work with pread as opposed to not work at all without pread? Yes, it does take the pread() code path (I verified that by adding debug log statements), but it doesn't count them as hedged reads. -- To view, visit http://gerrit.cloudera.org:8080/9966 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I48fe80dfd9a1ed68a8f2b7038e5f42b5a3df3baa Gerrit-Change-Number: 9966 Gerrit-PatchSet: 2 Gerrit-Owner: Sailesh Mukil <sail...@cloudera.com> Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com> Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Comment-Date: Tue, 22 May 2018 18:37:53 +0000 Gerrit-HasComments: No