Tim Armstrong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10532 )
Change subject: IMPALA-6953: part 1: clean up DiskIoMgr ...................................................................... IMPALA-6953: part 1: clean up DiskIoMgr There should be no behavioural changes as a result of this refactoring. Make DiskQueue an encapsulated class. Remove friend classes where possible, either by using public methods or moving code between classes. Move method into protected in some cases. Split GetNextRequestRange() into two methods that operate on DiskQueue and RequestContext state. The methods belong to the respective classes. Reduce transitive #include dependencies to hopefully help with build time. Testing: Ran core tests. Change-Id: I5a6e393f8c01d10143cbac91108af37f6498c1b1 Reviewed-on: http://gerrit.cloudera.org:8080/10245 Reviewed-by: Tim Armstrong <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> Reviewed-on: http://gerrit.cloudera.org:8080/10532 Reviewed-by: Thomas Marshall <[email protected]> Tested-by: Thomas Marshall <[email protected]> --- 0 files changed, 0 insertions(+), 0 deletions(-) Approvals: Thomas Marshall: Looks good to me, approved; Verified Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/10532 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: 2.x Gerrit-MessageType: merged Gerrit-Change-Id: I5a6e393f8c01d10143cbac91108af37f6498c1b1 Gerrit-Change-Number: 10532 Gerrit-PatchSet: 2 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Thomas Marshall <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
