Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15276 )
Change subject: mvcc: clean up public API ...................................................................... mvcc: clean up public API A bunch of MvccManager methods were public but were used only in tests or via ScopedTransactions. This makes such methods private, and updates most existing MVCC-related tests to use ScopedTransactions. There are no major functional changes in this patch. Change-Id: Ic4e8b2b5ba745b13a0ecb626473642f9d94df85f Reviewed-on: http://gerrit.cloudera.org:8080/15276 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/tablet/diff_scan-test.cc M src/kudu/tablet/mvcc-test.cc M src/kudu/tablet/mvcc.cc M src/kudu/tablet/mvcc.h 4 files changed, 227 insertions(+), 247 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/15276 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ic4e8b2b5ba745b13a0ecb626473642f9d94df85f Gerrit-Change-Number: 15276 Gerrit-PatchSet: 3 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
