Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9097 )
Change subject: IMPALA-4835: prerequisite buffer pool changes ...................................................................... IMPALA-4835: prerequisite buffer pool changes The scanner/buffer pool changes will have different scanner threads sharing the same buffer pool client. This requires that the AllocateBuffer() API is safe to call concurrently from different threads, which was true previously but not documented or tested. This updates the comments and adds a couple of tests. Change-Id: I8f2196722df59f2d367787c0550058022e296e24 Reviewed-on: http://gerrit.cloudera.org:8080/9097 Reviewed-by: Tim Armstrong <tarmstr...@cloudera.com> Tested-by: Impala Public Jenkins --- M be/src/runtime/bufferpool/buffer-pool-test.cc M be/src/runtime/bufferpool/buffer-pool.h 2 files changed, 145 insertions(+), 6 deletions(-) Approvals: Tim Armstrong: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/9097 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I8f2196722df59f2d367787c0550058022e296e24 Gerrit-Change-Number: 9097 Gerrit-PatchSet: 5 Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Bikramjeet Vig <bikramjeet....@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Michael Ho <k...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>