impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tim Armstrong (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-3202: variable-length scratch file ranges
Date Wed, 04 Jan 2017 21:24:51 GMT
Tim Armstrong has uploaded a new patch set (#4).

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/4
-- 
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: 4
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Tim Armstrong <tarmstrong@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dhecht@cloudera.com>

Mime
View raw message