[kudu-CR] WIP: tie log retention to consensus watermarks

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

[kudu-CR] WIP: 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 (#3). Change subject: WIP: tie log retention to consensus watermarks .. WIP: tie log

[kudu-CR] WIP: tie log retention to consensus watermarks

2016-09-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: WIP: tie log retention to consensus watermarks .. Patch Set 1: ok, new patch is largely cleaned up, though still planning on adding a test or two. Worth

[kudu-CR] WIP: 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 (#2). Change subject: WIP: tie log retention to consensus watermarks .. WIP: tie log

[kudu-CR] WIP: tie log retention to consensus watermarks

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

[kudu-CR] WIP: tie log retention to consensus watermarks

2016-09-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: WIP: tie log retention to consensus watermarks .. Patch Set 1: (3 comments) just responding to comments. will work on the TODOs in the patch and add tests.

[kudu-CR] WIP: tie log retention to consensus watermarks

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

[kudu-CR] WIP: tie log retention to consensus watermarks

2016-08-30 Thread Todd Lipcon (Code Review)
Hello Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4177 to review the following change. Change subject: WIP: tie log retention to consensus watermarks .. WIP: tie log