Jean-Daniel Cryans has posted comments on this change.

Change subject: Integrate the result tracker with writes

Patch Set 25:


Just nits.
File src/kudu/tablet/transactions/

PS25, Line 210: it's
nit: its

PS25, Line 268: er.");
Any more info that we can add here?
File src/kudu/tablet/transactions/transaction_driver.h:

PS25, Line 124: client originated
nit: client-originated

PS25, Line 124: a
nit: remove

PS25, Line 149: to to 

PS25, Line 155: it's
nit: its

PS25, Line 156: requests
nit: request? same for the other "requests" on that line.

PS25, Line 157:  

PS25, Line 168: and
nit: remove that 'and'

PS25, Line 169: received
nit: receive

PS25, Line 176: request were
nit: either request should be plural or were should be was.

PS25, Line 302: drivers
nit: driver
File src/kudu/tablet/transactions/write_transaction.h:

PS25, Line 192: pointers to the rpc context, request and response, lifecyle
              :   // is managed by the rpc subsystem.
Since you're here, mind rewriting this sentence? Start with upper case, there's 
an odd "and" in the middle, and it is "is managed" when talking about multiple 

PS25, Line 194: Request
nit: 'request_'

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I1fa2f8db33653960f4749237b8993baba0929893
Gerrit-PatchSet: 25
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <>
Gerrit-Reviewer: Adar Dembo <>
Gerrit-Reviewer: David Ribeiro Alves <>
Gerrit-Reviewer: Jean-Daniel Cryans <>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <>
Gerrit-Reviewer: Todd Lipcon <>
Gerrit-HasComments: Yes

Reply via email to