[kudu-CR] KUDU-636. Use Arena for EncodedKeys

2020-07-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/16162 ) Change subject: KUDU-636. Use Arena for EncodedKeys .. Patch Set 4: Verified+1 Test failure looks unrelated -- To view, visit

[kudu-CR] KUDU-636. Use Arena for EncodedKeys

2020-07-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has removed a vote on this change. Change subject: KUDU-636. Use Arena for EncodedKeys .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/16162 To unsubscribe, visit

[kudu-CR] KUDU-636. Use Arena for EncodedKeys

2020-07-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16162 ) Change subject: KUDU-636. Use Arena for EncodedKeys .. KUDU-636. Use Arena for EncodedKeys This updates EncodedKeyBuilder,

[kudu-CR] KUDU-636. Use Arena for EncodedKeys

2020-07-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16162 ) Change subject: KUDU-636. Use Arena for EncodedKeys .. Patch Set 4: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-636. Use Arena for EncodedKeys

2020-07-10 Thread Todd Lipcon (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16162 to look at the new patch set (#4). Change subject: KUDU-636. Use Arena for EncodedKeys ..

[kudu-CR] KUDU-636. Use Arena for EncodedKeys

2020-07-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16162 ) Change subject: KUDU-636. Use Arena for EncodedKeys .. Patch Set 3: Code-Review+2 (1 comment)

[kudu-CR] KUDU-636. Use Arena for EncodedKeys

2020-07-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/16162 ) Change subject: KUDU-636. Use Arena for EncodedKeys .. Patch Set 2: (2 comments)

[kudu-CR] KUDU-636. Use Arena for EncodedKeys

2020-07-10 Thread Todd Lipcon (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16162 to look at the new patch set (#3). Change subject: KUDU-636. Use Arena for EncodedKeys ..

[kudu-CR] KUDU-636. Use Arena for EncodedKeys

2020-07-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16162 ) Change subject: KUDU-636. Use Arena for EncodedKeys .. Patch Set 2: Code-Review+2 (2 comments) Looks good overall, just one style

[kudu-CR] KUDU-636. Use Arena for EncodedKeys

2020-07-09 Thread Todd Lipcon (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16162 to look at the new patch set (#2). Change subject: KUDU-636. Use Arena for EncodedKeys ..

[kudu-CR] KUDU-636. Use Arena for EncodedKeys

2020-07-09 Thread Todd Lipcon (Code Review)
Hello Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/16162 to review the following change. Change subject: KUDU-636. Use Arena for EncodedKeys .. KUDU-636. Use Arena for