Adar Dembo has posted comments on this change.

Change subject: KUDU-1933. consensus: Fix integer overflow in log index
......................................................................


Patch Set 1:

(1 comment)

Just passing through out of interest, not a real review.

http://gerrit.cloudera.org:8080/#/c/6376/1/src/kudu/consensus/opid_util.cc
File src/kudu/consensus/opid_util.cc:

Line 155: OpId MakeOpId(int64 term, int64 index) {
int64_t


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I284edbde51dc50fb2f98acc83cdcc3891d37863f
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mike Percy <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: Yes

Reply via email to