You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Tim Armstrong (Code Review)" <ge...@cloudera.org> on 2017/01/04 20:07:36 UTC

[Impala-ASF-CR] IMPALA-3202: variable-length scratch file ranges

Tim Armstrong has uploaded a new patch set (#3).

Change subject: IMPALA-3202: variable-length scratch file ranges
......................................................................

IMPALA-3202: variable-length scratch file ranges

This uses a simple approach where scratch ranges are managed in
power-of-two size classes and we don't attempt to coalesce or
split the ranges to move them into different size classes. Thus
this does not optimally reuse space if a query spills a variety
of page sizes, but improving this may not be worth the added
complexity.

Testing:
Extended tmp-file-mgr-test to exercise the variable scratch range
support. We will get system test coverage once this is used by the
new buffer pool.

Change-Id: Ic0ad84493c2c93a5602c404a83c718f25ea25575
---
M be/src/runtime/buffered-block-mgr.cc
M be/src/runtime/tmp-file-mgr-test.cc
M be/src/runtime/tmp-file-mgr.cc
M be/src/runtime/tmp-file-mgr.h
4 files changed, 95 insertions(+), 94 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/97/5597/3
-- 
To view, visit http://gerrit.cloudera.org:8080/5597
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic0ad84493c2c93a5602c404a83c718f25ea25575
Gerrit-PatchSet: 3
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Tim Armstrong <ta...@cloudera.com>