David Ribeiro Alves has submitted this change and it was merged.

Change subject: Don't default 'request' to null in WriteTransactionState's ctor
......................................................................


Don't default 'request' to null in WriteTransactionState's ctor

There is no case in which we start a WriteTransactionState without
a request, so there is no point in defaulting it to null.

Change-Id: I2b6d4e36f392e233e8afb66beb0221b914687486
Reviewed-on: http://gerrit.cloudera.org:8080/3290
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <a...@cloudera.com>
---
M src/kudu/tablet/transactions/write_transaction.cc
M src/kudu/tablet/transactions/write_transaction.h
2 files changed, 4 insertions(+), 9 deletions(-)

Approvals:
  Adar Dembo: Looks good to me, approved
  Kudu Jenkins: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2b6d4e36f392e233e8afb66beb0221b914687486
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <david.al...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <david.al...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to