[kudu-CR] Allow for reserving disk space for non-Kudu processes

2016-05-26 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3135 to look at the new patch set (#3). Change subject: Allow for reserving disk space for non-Kudu processes .. Allow

[kudu-CR] Don't use InMemoryEnv in deltafile-test

2016-05-26 Thread Mike Percy (Code Review)
Mike Percy has uploaded a new change for review. http://gerrit.cloudera.org:8080/3235 Change subject: Don't use InMemoryEnv in deltafile-test .. Don't use InMemoryEnv in deltafile-test Get out the vote: #NeverMemEnv. This

[kudu-CR] KUDU-1444. Get resource metrics of a scan.

2016-05-26 Thread zhen.zhang (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3013 to look at the new patch set (#8). Change subject: KUDU-1444. Get resource metrics of a scan. .. KUDU-1444. Get

[kudu-CR] Allow for reserving disk space for non-Kudu processes

2016-05-26 Thread Mike Percy (Code Review)
Hello Adar Dembo, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3135 to look at the new patch set (#2). Change subject: Allow for reserving disk space for non-Kudu processes

[kudu-CR] Make BuildLog() return Status

2016-05-26 Thread Mike Percy (Code Review)
Hello Adar Dembo, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3134 to look at the new patch set (#2). Change subject: Make BuildLog() return Status .. Make

[kudu-CR] log: Mark allocation finished even if allocation had an error

2016-05-26 Thread Mike Percy (Code Review)
Hello Adar Dembo, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3234 to review the following change. Change subject: log: Mark allocation finished even if allocation had an error

[kudu-CR] fix compile error when compiling column_predicate-test.cc In env : boost 1.57, centos 6.5, compile failed because of : "operator<<: cannot bind lvalue to 'std::basic_ostream&&'" wr

2016-05-26 Thread song bruce zhang (Code Review)
song bruce zhang has uploaded a new change for review. http://gerrit.cloudera.org:8080/3233 Change subject: fix compile error when compiling column_predicate-test.cc In env : boost 1.57, centos 6.5, compile failed because of : "operator<<: cannot bind lvalue to 'std::basic_ostream&&'"

[kudu-CR] KUDU-1470 Exceptions on getting a column value should return the column name not the column number

2016-05-26 Thread Ted Malaska (Code Review)
Ted Malaska has posted comments on this change. Change subject: KUDU-1470 Exceptions on getting a column value should return the column name not the column number .. Patch Set 2: I made a second patch because I noticed my

[kudu-CR] KUDU-1470 Exceptions on getting a column value should return the column name not the column number

2016-05-26 Thread Ted Malaska (Code Review)
Ted Malaska has uploaded a new patch set (#2). Change subject: KUDU-1470 Exceptions on getting a column value should return the column name not the column number .. KUDU-1470 Exceptions on getting a column value should return

[kudu-CR] KUDU-1470 Exceptions on getting a column value should return the column name not the column number

2016-05-26 Thread Ted Malaska (Code Review)
Ted Malaska has uploaded a new change for review. http://gerrit.cloudera.org:8080/3231 Change subject: KUDU-1470 Exceptions on getting a column value should return the column name not the column number .. KUDU-1470

[kudu-CR] Kudu 0.9.0 release notes edit

2016-05-26 Thread Misty Stanley-Jones (Code Review)
Misty Stanley-Jones has posted comments on this change. Change subject: Kudu 0.9.0 release notes edit .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/3176/4/docs/installation.adoc File docs/installation.adoc:

[kudu-CR] Kudu 0.9.0 release notes edit

2016-05-26 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3176 to look at the new patch set (#4). Change subject: Kudu 0.9.0 release notes edit .. Kudu 0.9.0 release notes edit

[kudu-CR](branch-0.9.x) Remove default table partitioning

2016-05-26 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: Remove default table partitioning .. Remove default table partitioning This commit removes the current default of creating tables with range partitioning

[kudu-CR](branch-0.9.x) Remove default table partitioning

2016-05-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Remove default table partitioning .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3229 To unsubscribe, visit

[kudu-CR](branch-0.9.x) Remove default table partitioning

2016-05-26 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded a new change for review. http://gerrit.cloudera.org:8080/3229 Change subject: Remove default table partitioning .. Remove default table partitioning This commit removes the current default of

[kudu-CR] Document advice about max columns and record size

2016-05-26 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Document advice about max columns and record size .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2778 To unsubscribe, visit

[kudu-CR] Non-covering Range Partitions design doc

2016-05-26 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Non-covering Range Partitions design doc .. Patch Set 11: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2772 To unsubscribe, visit

[kudu-CR] Non-covering Range Partitions design doc

2016-05-26 Thread Dan Burkert (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2772 to look at the new patch set (#11). Change subject: Non-covering Range Partitions design doc ..

[kudu-CR] Remove default table partitioning

2016-05-26 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Remove default table partitioning .. Patch Set 17: (1 comment) http://gerrit.cloudera.org:8080/#/c/3131/17/docs/schema_design.adoc File docs/schema_design.adoc:

[kudu-CR] Remove default table partitioning

2016-05-26 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3131 to look at the new patch set (#17). Change subject: Remove default table partitioning .. Remove default table

[kudu-CR] Remove default table partitioning

2016-05-26 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Remove default table partitioning .. Patch Set 15: (1 comment) http://gerrit.cloudera.org:8080/#/c/3131/15/docs/schema_design.adoc File docs/schema_design.adoc:

[kudu-CR] Remove default table partitioning

2016-05-26 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3131 to look at the new patch set (#16). Change subject: Remove default table partitioning .. Remove default table

[kudu-CR] Remove default table partitioning

2016-05-26 Thread Misty Stanley-Jones (Code Review)
Misty Stanley-Jones has posted comments on this change. Change subject: Remove default table partitioning .. Patch Set 13: (3 comments) http://gerrit.cloudera.org:8080/#/c/3131/13/docs/release_notes.adoc File

[kudu-CR] Remove default table partitioning

2016-05-26 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Remove default table partitioning .. Patch Set 13: (1 comment) http://gerrit.cloudera.org:8080/#/c/3131/13/docs/schema_design.adoc File docs/schema_design.adoc:

[kudu-CR] Remove default table partitioning

2016-05-26 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Remove default table partitioning .. Patch Set 13: (5 comments) http://gerrit.cloudera.org:8080/#/c/3131/13/docs/release_notes.adoc File docs/release_notes.adoc:

[kudu-CR] Remove default table partitioning

2016-05-26 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3131 to look at the new patch set (#15). Change subject: Remove default table partitioning .. Remove default table

[kudu-CR] Kudu 0.9.0 release notes edit

2016-05-26 Thread Misty Stanley-Jones (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3176 to look at the new patch set (#2). Change subject: Kudu 0.9.0 release notes edit .. Kudu 0.9.0 release notes edit

[kudu-CR](branch-0.9.x) KUDU-749 (part 2): avoid O(n^2) behavior when compacting deltas

2016-05-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-749 (part 2): avoid O(n^2) behavior when compacting deltas .. KUDU-749 (part 2): avoid O(n^2) behavior when compacting deltas When handling a

[kudu-CR](branch-0.9.x) KUDU-749 (part 2): avoid O(n^2) behavior when compacting deltas

2016-05-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-749 (part 2): avoid O(n^2) behavior when compacting deltas .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3224 To

[kudu-CR] Document advice about max columns and record size

2016-05-26 Thread Misty Stanley-Jones (Code Review)
Misty Stanley-Jones has posted comments on this change. Change subject: Document advice about max columns and record size .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/2778/3/docs/schema_design.adoc File

[kudu-CR] Document advice about max columns and record size

2016-05-26 Thread Misty Stanley-Jones (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2778 to look at the new patch set (#4). Change subject: Document advice about max columns and record size .. Document

[kudu-CR] KUDU-1444. Get resource metrics of a scan.

2016-05-26 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1444. Get resource metrics of a scan. .. Patch Set 7: (3 comments) http://gerrit.cloudera.org:8080/#/c/3013/7/src/kudu/client/client-test.cc File

[kudu-CR](gh-pages) Add support for anchors.js to get permalinks on all headers

2016-05-26 Thread Misty Stanley-Jones (Code Review)
Misty Stanley-Jones has uploaded a new patch set (#4). Change subject: Add support for anchors.js to get permalinks on all headers .. Add support for anchors.js to get permalinks on all headers Change-Id:

[kudu-CR] Remove default table partitioning

2016-05-26 Thread Misty Stanley-Jones (Code Review)
Misty Stanley-Jones has posted comments on this change. Change subject: Remove default table partitioning .. Patch Set 13: (5 comments) http://gerrit.cloudera.org:8080/#/c/3131/13/docs/release_notes.adoc File

[kudu-CR] Remove default table partitioning

2016-05-26 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3131 to look at the new patch set (#14). Change subject: Remove default table partitioning .. Remove default table

[kudu-CR] Reduce verbosity of Java logs

2016-05-26 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: Reduce verbosity of Java logs .. Reduce verbosity of Java logs This commit changes the logging format in order to make it more compact. Date and

[kudu-CR] Remove default table partitioning

2016-05-26 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3131 to look at the new patch set (#13). Change subject: Remove default table partitioning .. Remove default table

[kudu-CR] Reduce verbosity of Java logs

2016-05-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Reduce verbosity of Java logs .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3203 To unsubscribe, visit

[kudu-CR] Reduce verbosity of Java logs

2016-05-26 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3203 to look at the new patch set (#3). Change subject: Reduce verbosity of Java logs .. Reduce verbosity of Java logs

[kudu-CR] Remove default table partitioning

2016-05-26 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Remove default table partitioning .. Patch Set 9: (2 comments)

[kudu-CR] Remove default table partitioning

2016-05-26 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3131 to look at the new patch set (#12). Change subject: Remove default table partitioning .. Remove default table

[kudu-CR](branch-0.9.x) ksck: usability improvements in error messages

2016-05-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: ksck: usability improvements in error messages .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3226 To unsubscribe, visit

[kudu-CR](branch-0.9.x) ksck: usability improvements in error messages

2016-05-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: ksck: usability improvements in error messages .. ksck: usability improvements in error messages - print the address of tablet servers that fail

[kudu-CR] KUDU-1444. Get resource metrics of a scan.

2016-05-26 Thread zhen.zhang (Code Review)
zhen.zhang has posted comments on this change. Change subject: KUDU-1444. Get resource metrics of a scan. .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/3013/7/src/kudu/client/client-test.cc File

[kudu-CR] KUDU-1444. Get resource metrics of a scan.

2016-05-26 Thread zhen.zhang (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3013 to look at the new patch set (#7). Change subject: KUDU-1444. Get resource metrics of a scan. .. KUDU-1444. Get

[kudu-CR] KUDU-1444. Get resource metrics of a scan.

2016-05-26 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1444. Get resource metrics of a scan. .. Patch Set 6: OK. The code now looks good, but missing a test. Sorry, I should have mentioned that previously. Maybe