[kudu-CR] Separate tablet MM ops into separate .cc file

2016-10-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Separate tablet MM ops into separate .cc file .. Separate tablet MM ops into separate .cc file tablet.cc is a monster, no need for this in there. Plus,

[kudu-CR] Separate tablet MM ops into separate .cc file

2016-10-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Separate tablet MM ops into separate .cc file .. Patch Set 6: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4362 To unsubscribe,

[kudu-CR] Separate tablet MM ops into separate .cc file

2016-10-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Separate tablet MM ops into separate .cc file .. Patch Set 6: Code-Review+2 I didn't run a line-by-line comparison of your copy paste so I assume it's fine

[kudu-CR] Separate tablet MM ops into separate .cc file

2016-10-04 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4362 to look at the new patch set (#6). Change subject: Separate tablet MM ops into separate .cc file .. Separate

[kudu-CR] Separate tablet MM ops into separate .cc file

2016-09-17 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Separate tablet MM ops into separate .cc file .. Patch Set 4: This is just moving code around, so I would rather not make modifications. I didn't change anything.

[kudu-CR] Separate tablet MM ops into separate .cc file

2016-09-10 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4362 to look at the new patch set (#2). Change subject: Separate tablet MM ops into separate .cc file .. Separate

[kudu-CR] Separate tablet MM ops into separate .cc file

2016-09-09 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Separate tablet MM ops into separate .cc file .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4362/1/src/kudu/tablet/tablet_mm_ops.cc File

[kudu-CR] Separate tablet MM ops into separate .cc file

2016-09-09 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Separate tablet MM ops into separate .cc file .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3324/ -- To view, visit