Tim Armstrong has posted comments on this change. Change subject: IMPALA-3276: consistently handle pin failure in BTS::PrepareForRead() ......................................................................
Patch Set 1: This passes a private exhaustive build. I actually ran into this problem when converting the code to use the new buffer pool: I think the changed interface makes sense even if reservations are more guaranteed. -- To view, visit http://gerrit.cloudera.org:8080/2684 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iacf1aba50d79b01acdc7269dc23f07b8c19a151a Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-HasComments: No
