[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-25 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10926 ) Change subject: memrowset: support iteration with snap_to_exclude .. memrowset: support iteration with snap_to_exclude Following

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-25 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/10926 ) Change subject: memrowset: support iteration with snap_to_exclude .. Patch Set 8: Code-Review+2 I'm good with this if Grant is. -- To v

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-24 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/10926 ) Change subject: memrowset: support iteration with snap_to_exclude .. Patch Set 8: Code-Review+1 much easier for me to follow now, thanks

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-24 Thread Adar Dembo (Code Review)
Hello Tidy Bot, Mike Percy, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10926 to look at the new patch set (#8). Change subject: memrowset: support iteration with snap_to_exclude .

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-24 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10926 ) Change subject: memrowset: support iteration with snap_to_exclude .. Patch Set 7: (2 comments) http://gerrit.cloudera.org:8080/#/c/10926

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-24 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/10926 ) Change subject: memrowset: support iteration with snap_to_exclude .. Patch Set 7: (2 comments) http://gerrit.cloudera.org:8080/#/c/1092

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-24 Thread Adar Dembo (Code Review)
Adar Dembo has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/10926 ) Change subject: memrowset: support iteration with snap_to_exclude .. Removed reviewer Kudu Jenkins with the following votes: * Ve

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-24 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10926 ) Change subject: memrowset: support iteration with snap_to_exclude .. Patch Set 7: Verified+1 Overriding Jenkins, unrelated Java test fail

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-23 Thread Adar Dembo (Code Review)
Hello Tidy Bot, Mike Percy, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10926 to look at the new patch set (#7). Change subject: memrowset: support iteration with snap_to_exclude .

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-23 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/10926 ) Change subject: memrowset: support iteration with snap_to_exclude .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/10926

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-22 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10926 ) Change subject: memrowset: support iteration with snap_to_exclude .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/c/10926

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-20 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/10926 ) Change subject: memrowset: support iteration with snap_to_exclude .. Patch Set 6: Code-Review+2 (2 comments) http://gerrit.cloudera.org

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-19 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/10926 ) Change subject: memrowset: support iteration with snap_to_exclude .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-19 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10926 ) Change subject: memrowset: support iteration with snap_to_exclude .. Patch Set 6: Verified+1 Overriding Jenkins, unrelated test failures.

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-19 Thread Adar Dembo (Code Review)
Adar Dembo has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/10926 ) Change subject: memrowset: support iteration with snap_to_exclude .. Removed reviewer Kudu Jenkins with the following votes: * Ve

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-19 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10926 ) Change subject: memrowset: support iteration with snap_to_exclude .. Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/10926

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-19 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/10926 ) Change subject: memrowset: support iteration with snap_to_exclude .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/10926/

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-19 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/10926 ) Change subject: memrowset: support iteration with snap_to_exclude .. Patch Set 5: (3 comments) Nice test! http://gerrit.cloudera.org:80

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-17 Thread Adar Dembo (Code Review)
Adar Dembo has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/10926 ) Change subject: memrowset: support iteration with snap_to_exclude .. Removed reviewer Kudu Jenkins with the following votes: * Ve

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-17 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10926 ) Change subject: memrowset: support iteration with snap_to_exclude .. Patch Set 5: Verified+1 Overriding Jenkins, unrelated test failure.

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-12 Thread Adar Dembo (Code Review)
Hello Tidy Bot, Mike Percy, Kudu Jenkins, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10926 to look at the new patch set (#4). Change subject: memrowset: support iteration with snap_to_exclude ...

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-12 Thread Adar Dembo (Code Review)
Hello Tidy Bot, Mike Percy, Kudu Jenkins, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10926 to look at the new patch set (#3). Change subject: memrowset: support iteration with snap_to_exclude ...

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-11 Thread Adar Dembo (Code Review)
Hello Tidy Bot, Mike Percy, Kudu Jenkins, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10926 to look at the new patch set (#2). Change subject: memrowset: support iteration with snap_to_exclude ...

[kudu-CR] memrowset: support iteration with snap to exclude

2018-07-11 Thread Adar Dembo (Code Review)
Hello Mike Percy, Grant Henke, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/10926 to review the following change. Change subject: memrowset: support iteration with snap_to_exclude ...