[kudu-CR] Allow to set RequestId in the RPC RequestHeader

2016-05-25 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3179 to look at the new patch set (#3). Change subject: Allow to set RequestId in the RPC RequestHeader .. Allow to set

[kudu-CR] Remove default table partitioning

2016-05-25 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 (#10). Change subject: Remove default table partitioning .. Remove default table

[kudu-CR] Remove default table partitioning

2016-05-25 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 (#11). Change subject: Remove default table partitioning .. Remove default table

[kudu-CR](branch-0.9.x) python: add support for specifying partitioning

2016-05-25 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: python: add support for specifying partitioning .. python: add support for specifying partitioning This adds support for range and hash

[kudu-CR] Reduce verbosity of Java logs

2016-05-25 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Reduce verbosity of Java logs .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3203/1/java/kudu-client/src/test/resources/log4j.properties File

[kudu-CR](branch-0.9.x) python: add support for specifying partitioning

2016-05-25 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: python: add support for specifying partitioning .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3218 To unsubscribe, visit

[kudu-CR] Dependency on Hadoop test classes causes Javadoc warnings and build failure

2016-05-25 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Dependency on Hadoop test classes causes Javadoc warnings and build failure .. Dependency on Hadoop test classes causes Javadoc warnings and build

[kudu-CR] python: add support for specifying partitioning

2016-05-25 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: python: add support for specifying partitioning .. python: add support for specifying partitioning This adds support for range and hash partitioning when

[kudu-CR] Reduce verbosity of Java logs

2016-05-25 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 (#2). Change subject: Reduce verbosity of Java logs .. Reduce verbosity of Java logs

[kudu-CR](branch-0.9.x) python: add support for specifying partitioning

2016-05-25 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded a new change for review. http://gerrit.cloudera.org:8080/3218 Change subject: python: add support for specifying partitioning .. python: add support for specifying partitioning This adds support for

[kudu-CR] python: add support for specifying partitioning

2016-05-25 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: python: add support for specifying partitioning .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3196 To unsubscribe, visit

[kudu-CR] KUDU-1267: Calling getString() on RowResult for wrong type gave weird exception

2016-05-25 Thread Ted Malaska (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3102 to look at the new patch set (#6). Change subject: KUDU-1267: Calling getString() on RowResult for wrong type gave weird exception

[kudu-CR] Allow to set RequestId in the RPC RequestHeader

2016-05-25 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to set RequestId in the RPC RequestHeader .. Patch Set 3: Verified+1 unrelated java failure -- To view, visit http://gerrit.cloudera.org:8080/3179

[kudu-CR] Reduce verbosity of Java logs

2016-05-25 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Reduce verbosity of Java logs .. Patch Set 2: (1 comment)

[kudu-CR](branch-0.9.x) java: use truncated randomized exponential backoff for retries

2016-05-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: java: use truncated randomized exponential backoff for retries .. java: use truncated randomized exponential backoff for retries This changes the Java

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

2016-05-25 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 5: (7 comments) http://gerrit.cloudera.org:8080/#/c/3013/5/src/kudu/client/client.h File

[kudu-CR] Remove a stray WARNING left in deltafile.cc

2016-05-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Remove a stray WARNING left in deltafile.cc .. Patch Set 1: Code-Review+2 Going to self-+2 this one since it's trivial -- To view, visit

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

2016-05-25 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, Mike Percy, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3221 to look at the new patch set (#2). Change subject: KUDU-749 (part 2): avoid O(n^2) behavior when compacting deltas

[kudu-CR] KUDU-1267: Calling getString() on RowResult for wrong type gave weird exception

2016-05-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1267: Calling getString() on RowResult for wrong type gave weird exception .. Patch Set 6: Ted, your patch still isn't compiling. Take a look at the latest

[kudu-CR] Remove default table partitioning

2016-05-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Remove default table partitioning .. Patch Set 11: Python tests are still broken, and it looks like there's an RpcBenchmark failure too? Oh, and I think you

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

2016-05-25 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 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3221/2//COMMIT_MSG Commit

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

2016-05-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-749 (part 2): avoid O(n^2) behavior when compacting deltas .. Patch Set 2: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/3221/2//COMMIT_MSG

[kudu-CR] KUDU-1267: Calling getString() on RowResult for wrong type gave weird exception

2016-05-25 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: KUDU-1267: Calling getString() on RowResult for wrong type gave weird exception .. KUDU-1267: Calling getString() on RowResult for wrong type gave weird

[kudu-CR] ksck: usability improvements in error messages

2016-05-25 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/3225 To unsubscribe, visit

[kudu-CR](branch-0.9.x) Remove a stray WARNING left in deltafile.cc

2016-05-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/3212 Change subject: Remove a stray WARNING left in deltafile.cc .. Remove a stray WARNING left in deltafile.cc This was a debugging warning