[kudu-CR] compaction: Add additional validation in DeltaTracker

2017-02-07 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: compaction: Add additional validation in DeltaTracker .. compaction: Add additional validation in DeltaTracker This adds DEBUG-mode validation to

[kudu-CR] compaction: Add additional validation in DeltaTracker

2017-02-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: compaction: Add additional validation in DeltaTracker .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5919 To

[kudu-CR] compaction: Add additional validation in DeltaTracker

2017-02-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: compaction: Add additional validation in DeltaTracker .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5919 To unsubscribe,

[kudu-CR] compaction: Add additional validation in DeltaTracker

2017-02-06 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5919 to look at the new patch set (#3). Change subject: compaction: Add additional validation in DeltaTracker ..

[kudu-CR] compaction: Add additional validation in DeltaTracker

2017-02-06 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5919 to look at the new patch set (#2). Change subject: compaction: Add additional validation in DeltaTracker ..

[kudu-CR] compaction: Add additional validation in DeltaTracker

2017-02-06 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: compaction: Add additional validation in DeltaTracker .. Patch Set 1: (3 comments) > Worth adding a test? I didn't change any functionality, I simply added new

[kudu-CR] compaction: Add additional validation in DeltaTracker

2017-02-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: compaction: Add additional validation in DeltaTracker .. Patch Set 1: (3 comments)

[kudu-CR] compaction: Add additional validation in DeltaTracker

2017-02-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: compaction: Add additional validation in DeltaTracker .. Patch Set 1: Worth adding a test? -- To view, visit http://gerrit.cloudera.org:8080/5919 To

[kudu-CR] compaction: Add additional validation in DeltaTracker

2017-02-06 Thread Mike Percy (Code Review)
Hello David Ribeiro Alves, Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5919 to review the following change. Change subject: compaction: Add additional validation in DeltaTracker