Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14253 )
Change subject: [tablet] Remove used state member ...................................................................... [tablet] Remove used state member Each type of transaction subclass (i.e. WriteTransaction and AlterSchemaTransaction) uses its own state member, so the state member of the base class is useless Change-Id: I7d18063438905a5938cd6ae80f21179f0d6b0dd5 Reviewed-on: http://gerrit.cloudera.org:8080/14253 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/tablet/transactions/alter_schema_transaction.cc M src/kudu/tablet/transactions/transaction.cc M src/kudu/tablet/transactions/transaction.h M src/kudu/tablet/transactions/transaction_tracker-test.cc M src/kudu/tablet/transactions/write_transaction.cc 5 files changed, 10 insertions(+), 12 deletions(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/14253 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I7d18063438905a5938cd6ae80f21179f0d6b0dd5 Gerrit-Change-Number: 14253 Gerrit-PatchSet: 2 Gerrit-Owner: Anonymous Coward <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
