[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement .. KUDU-798 (part 4) Add a TimeManager to manage safe time advancement This p

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement .. Patch Set 18: Code-Review+2 keeping todd's previous +2 since I fixed the nit -- T

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread David Ribeiro Alves (Code Review)
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5300 to look at the new patch set (#18). Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement ..

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement .. Patch Set 16: (2 comments) http://gerrit.cloudera.org:8080/#/c/5300/16/src/kudu/co

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement .. Patch Set 16: (1 comment) http://gerrit.cloudera.org:8080/#/c/5300/16/src/kudu/consensus

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement .. Patch Set 15: (1 comment) http://gerrit.cloudera.org:8080/#/c/5300/15/src/kudu/consensus

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement .. Patch Set 16: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/5300/16/src/ku

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement .. Patch Set 15: (1 comment) http://gerrit.cloudera.org:8080/#/c/5300/15/src/kudu/con

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement .. Patch Set 15: (1 comment) http://gerrit.cloudera.org:8080/#/c/5300/15/src/kudu/con

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement .. Patch Set 15: (1 comment) http://gerrit.cloudera.org:8080/#/c/5300/15/src/kudu/consensus

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5300 to look at the new patch set (#16). Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement ...

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement .. Patch Set 15: (14 comments) http://gerrit.cloudera.org:8080/#/c/5300/15/src/kudu/c

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement .. Patch Set 15: (15 comments) http://gerrit.cloudera.org:8080/#/c/5300/15/src/kudu/consens

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement .. Patch Set 14: (12 comments) http://gerrit.cloudera.org:8080/#/c/5300/14/src/kudu/c

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5300 to look at the new patch set (#15). Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement ...

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement .. Patch Set 14: (12 comments) http://gerrit.cloudera.org:8080/#/c/5300/14/src/kudu/consensus

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5300 to look at the new patch set (#14). Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement ...

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5300 to look at the new patch set (#13). Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement ...

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5300 to look at the new patch set (#12). Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement ...

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5300 to look at the new patch set (#11). Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement ...

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-04 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5300 to look at the new patch set (#10). Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement ...

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-04 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5300 to look at the new patch set (#9). Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement