[kudu-CR] generic iterators: switch MergeIterator to intrusive list of states

2019-04-09 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12944 ) Change subject: generic_iterators: switch MergeIterator to intrusive list of states .. generic_iterators: switch MergeIterator

[kudu-CR] generic iterators: switch MergeIterator to intrusive list of states

2019-04-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/12944 ) Change subject: generic_iterators: switch MergeIterator to intrusive list of states .. Patch Set 4: Code-Review+2 -- To view, visit

[kudu-CR] generic iterators: switch MergeIterator to intrusive list of states

2019-04-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12944 ) Change subject: generic_iterators: switch MergeIterator to intrusive list of states .. Patch Set 3: > Patch Set 3: > > I experimented

[kudu-CR] generic iterators: switch MergeIterator to intrusive list of states

2019-04-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12944 ) Change subject: generic_iterators: switch MergeIterator to intrusive list of states .. Patch Set 3: I experimented with the different

[kudu-CR] generic iterators: switch MergeIterator to intrusive list of states

2019-04-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12944 ) Change subject: generic_iterators: switch MergeIterator to intrusive list of states .. Patch Set 1: (1 comment)

[kudu-CR] generic iterators: switch MergeIterator to intrusive list of states

2019-04-09 Thread Adar Dembo (Code Review)
Hello Mike Percy, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12944 to look at the new patch set (#2). Change subject: generic_iterators: switch MergeIterator to intrusive list of states

[kudu-CR] generic iterators: switch MergeIterator to intrusive list of states

2019-04-09 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/12944 ) Change subject: generic_iterators: switch MergeIterator to intrusive list of states .. Patch Set 1: (1 comment)

[kudu-CR] generic iterators: switch MergeIterator to intrusive list of states

2019-04-05 Thread Adar Dembo (Code Review)
Hello Mike Percy, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/12944 to review the following change. Change subject: generic_iterators: switch MergeIterator to intrusive list of states