Hello Bikramjeet Vig,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9097
to look at the new patch set (#2).
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
---
M be/src/runtime/bufferpool/buffer-pool-test.cc
M be/src/runtime/bufferpool/buffer-pool.h
2 files changed, 142 insertions(+), 4 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/97/9097/2
--
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: newpatchset
Gerrit-Change-Id: I8f2196722df59f2d367787c0550058022e296e24
Gerrit-Change-Number: 9097
Gerrit-PatchSet: 2
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Bikramjeet Vig <[email protected]>
