Todd Lipcon has submitted this change and it was merged. Change subject: Separate tablet MM ops into separate .cc file ......................................................................
Separate tablet MM ops into separate .cc file tablet.cc is a monster, no need for this in there. Plus, these classes already have their own header file Change-Id: Ie40475fdb9f5da6a7300661d5149509da990939d Reviewed-on: http://gerrit.cloudera.org:8080/4362 Reviewed-by: Jean-Daniel Cryans <jdcry...@apache.org> Reviewed-by: Todd Lipcon <t...@apache.org> Tested-by: Todd Lipcon <t...@apache.org> --- M src/kudu/tablet/CMakeLists.txt M src/kudu/tablet/tablet.cc A src/kudu/tablet/tablet_mm_ops.cc M src/kudu/tablet/tablet_mm_ops.h 4 files changed, 245 insertions(+), 213 deletions(-) Approvals: Jean-Daniel Cryans: Looks good to me, approved Todd Lipcon: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/4362 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie40475fdb9f5da6a7300661d5149509da990939d Gerrit-PatchSet: 7 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org>