[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 11: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3304 To unsubs

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. KUDU-1398 CFile index blocks can store shortest separating prefix (No changes: resub

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-23 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 11: Build Started http://104.196.14.100/job/kudu-gerrit/1973/ -- To view, visit

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-22 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 9: > Looks like a small compile error here :( whoops...was too hasty... -- To

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-22 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 10: Build Started http://104.196.14.100/job/kudu-gerrit/1948/ -- To view, visit

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-22 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3304 to look at the new patch set (#10). Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 9: Looks like a small compile error here -- To view, visit http://gerrit.cloudera

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-22 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 8: (9 comments) http://gerrit.cloudera.org:8080/#/c/3304/8/src/kudu/cfile/binary

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-22 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3304 to look at the new patch set (#9). Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix ..

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-22 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 9: Build Started http://104.196.14.100/job/kudu-gerrit/1927/ -- To view, visit h

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 8: (9 comments) http://gerrit.cloudera.org:8080/#/c/3304/8/src/kudu/cfile/binary_d

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 8: Build Started http://104.196.14.100/job/kudu-gerrit/1870/ -- To view, visit h

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 7: Build Started http://104.196.14.100/job/kudu-gerrit/1869/ -- To view, visit h

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-20 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3304 to look at the new patch set (#7). Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix ..

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-20 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 6: > Sorry for the delay on this, Will. I was on PTO the last couple of > weeks.

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-20 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 4: Sorry for the delay on this, Will. I was on PTO the last couple of weeks. Hope

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 6: Build Started http://104.196.14.100/job/kudu-gerrit/1787/ -- To view, visit h

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-08 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3304 to look at the new patch set (#6). Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix ..

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-08 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 4: (15 comments) http://gerrit.cloudera.org:8080/#/c/3304/4//COMMIT_MSG Commit M

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-08 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3304 to look at the new patch set (#5). Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix ..

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/1786/ -- To view, visit h

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 4: (15 comments) http://gerrit.cloudera.org:8080/#/c/3304/4//COMMIT_MSG Commit Mes

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-06 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/1749/ -- To view, visit h

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-06 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3304 to look at the new patch set (#4). Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix ..

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-03 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/1739/ -- To view, visit h

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-03 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded a new change for review. http://gerrit.cloudera.org:8080/3304 Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. KUDU-1398 CFile index blocks can store shortest sep