[kudu-CR] MergeIterator: don't stop iterating on an empty block

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: MergeIterator: don't stop iterating on an empty block .. MergeIterator: don't stop iterating on an empty block The MergeIterator implementation was incorr

[kudu-CR] MergeIterator: don't stop iterating on an empty block

2016-08-15 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: MergeIterator: don't stop iterating on an empty block .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3976 To unsubscribe, visit ht

[kudu-CR] MergeIterator: don't stop iterating on an empty block

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: MergeIterator: don't stop iterating on an empty block .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3976/2/src/kudu/common/generic_iterators-tes

[kudu-CR] MergeIterator: don't stop iterating on an empty block

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: MergeIterator: don't stop iterating on an empty block .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2913/ -- To view, visit http://gerrit.

[kudu-CR] MergeIterator: don't stop iterating on an empty block

2016-08-15 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3976 to look at the new patch set (#4). Change subject: MergeIterator: don't stop iterating on an empty block .. MergeIt

[kudu-CR] MergeIterator: don't stop iterating on an empty block

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: MergeIterator: don't stop iterating on an empty block .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2909/ -- To view, visit http://gerrit.

[kudu-CR] MergeIterator: don't stop iterating on an empty block

2016-08-15 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: MergeIterator: don't stop iterating on an empty block .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3976/2/src/kudu/common/generic_iterators-test

[kudu-CR] MergeIterator: don't stop iterating on an empty block

2016-08-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: MergeIterator: don't stop iterating on an empty block .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2888/ -- To view, visit http://gerrit.

[kudu-CR] MergeIterator: don't stop iterating on an empty block

2016-08-14 Thread Todd Lipcon (Code Review)
Hello Anonymous Coward #80, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3976 to look at the new patch set (#2). Change subject: MergeIterator: don't stop iterating on an empty block .