Todd Lipcon has submitted this change and it was merged.

Change subject: tablet: refactor some common CHECKs into ApplyRowOperation()
......................................................................


tablet: refactor some common CHECKs into ApplyRowOperation()

The Insert and Mutate paths had more or less the same set of precondition
CHECKs. These methods are not called from anywhere except for
ApplyRowOperation(), so this just moves the checks.

Change-Id: I5564296bacc97f224fed6d21f0c073e88fc1fbaa
Reviewed-on: http://gerrit.cloudera.org:8080/3118
Reviewed-by: David Ribeiro Alves <[email protected]>
Tested-by: Kudu Jenkins
---
M src/kudu/tablet/tablet.cc
1 file changed, 6 insertions(+), 11 deletions(-)

Approvals:
  David Ribeiro Alves: Looks good to me, approved
  Kudu Jenkins: Verified



-- 
To view, visit http://gerrit.cloudera.org:8080/3118
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I5564296bacc97f224fed6d21f0c073e88fc1fbaa
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>

Reply via email to