[kudu-CR] Tie log retention to consensus watermarks

2016-09-09 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Tie log retention to consensus watermarks .. Tie log retention to consensus watermarks This changes the calculation of log retention to consult

[kudu-CR] Tie log retention to consensus watermarks

2016-09-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Tie log retention to consensus watermarks .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4177 To unsubscribe, visit

[kudu-CR] Tie log retention to consensus watermarks

2016-09-09 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4177 to look at the new patch set (#7). Change subject: Tie log retention to consensus watermarks

[kudu-CR] Tie log retention to consensus watermarks

2016-09-09 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Tie log retention to consensus watermarks .. Patch Set 7: Build Started http://104.196.14.100/job/kudu-gerrit/3307/ -- To view, visit

[kudu-CR] Tie log retention to consensus watermarks

2016-09-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Tie log retention to consensus watermarks .. Patch Set 6: Code-Review+1 (3 comments) A couple very minor nits but they are ignorable. LGTM

[kudu-CR] Tie log retention to consensus watermarks

2016-09-09 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Tie log retention to consensus watermarks .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4177 To unsubscribe, visit

[kudu-CR] Tie log retention to consensus watermarks

2016-09-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Tie log retention to consensus watermarks .. Patch Set 6: Build Started http://104.196.14.100/job/kudu-gerrit/3286/ -- To view, visit

[kudu-CR] Tie log retention to consensus watermarks

2016-09-08 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4177 to look at the new patch set (#6). Change subject: Tie log retention to consensus watermarks .. Tie log retention

[kudu-CR] Tie log retention to consensus watermarks

2016-09-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Tie log retention to consensus watermarks .. Patch Set 5: (6 comments) http://gerrit.cloudera.org:8080/#/c/4177/5//COMMIT_MSG Commit Message: Line 11:

[kudu-CR] Tie log retention to consensus watermarks

2016-09-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Tie log retention to consensus watermarks .. Patch Set 5: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/3268/ -- To view, visit

[kudu-CR] Tie log retention to consensus watermarks

2016-09-07 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4177 to look at the new patch set (#5). Change subject: Tie log retention to consensus watermarks .. Tie log retention

[kudu-CR] Tie log retention to consensus watermarks

2016-09-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Tie log retention to consensus watermarks .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/3265/ -- To view, visit