Dan Hecht has posted comments on this change.
Change subject: IMPALA-3202: variable-length scratch file ranges
......................................................................
Patch Set 4: Code-Review+2
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5597/4/be/src/runtime/tmp-file-mgr.cc
File be/src/runtime/tmp-file-mgr.cc:
Line 351: void TmpFileMgr::FileGroup::AddFreeRange(WriteHandle* handle) {
since it wouldn't be valid to use the WriteHandle once the range is added to the free list,
you could even rename this to "RecycleFileRange()" and make this take the unique_ptr and have
it do the reset. I'm also fine with the current code.
--
To view, visit http://gerrit.cloudera.org:8080/5597
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
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>
Gerrit-HasComments: Yes