[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-30 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 9: Thanks for the reviews everyone! -- To view,

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-30 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. KUDU-3108: fix invalid memory accesses in merge

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 8: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-28 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 8: (3 comments)

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-28 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Lieber-Dembo, Bankim Bhavsar, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16777 to look at the new patch set (#8). Change subject: KUDU-3108: fix invalid memory accesses in merge iterator

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-27 Thread Adar Lieber-Dembo (Code Review)
Adar Lieber-Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 7: Code-Review+2 (4 comments)

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-27 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 7: Verified+1 Test failure was hms_client-test

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-27 Thread Andrew Wong (Code Review)
Andrew Wong has removed a vote on this change. Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/16777 To

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-26 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Lieber-Dembo, Bankim Bhavsar, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16777 to look at the new patch set (#7). Change subject: KUDU-3108: fix invalid memory accesses in merge iterator

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-26 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 6: (1 comment)

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 6: (1 comment)

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-25 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 6: (1 comment)

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 6: (1 comment)

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-25 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 6: (3 comments)

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-25 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 6: (2 comments)

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-25 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 6: (1 comment)

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-25 Thread Adar Lieber-Dembo (Code Review)
Adar Lieber-Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 6: (2 comments)

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-24 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 6: (1 comment)

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-24 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 6: I seem to recall that adar had a simpler

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 6: Code-Review+2 Looks good to me. I guess it

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-24 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 6: (3 comments)

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-24 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Lieber-Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16777 to look at the new patch set (#6). Change subject: KUDU-3108: fix invalid memory accesses in merge iterator

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 5: Code-Review+1 (3 comments)

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-24 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 4: (8 comments)

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-24 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Lieber-Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16777 to look at the new patch set (#4). Change subject: KUDU-3108: fix invalid memory accesses in merge iterator

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 3: (1 comment)

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 3: (1 comment)

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-24 Thread Adar Lieber-Dembo (Code Review)
Adar Lieber-Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/16777 ) Change subject: KUDU-3108: fix invalid memory accesses in merge iterator .. Patch Set 3: (6 comments) Good find, thanks for this

[kudu-CR] KUDU-3108: fix invalid memory accesses in merge iterator

2020-11-23 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16777 to look at the new patch set (#2). Change subject: KUDU-3108: fix invalid memory accesses in merge iterator