[kudu-CR] [logging] fix logging init if linking in kudu client library

2018-07-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10946 Change subject: [logging] fix logging init if linking in kudu client library .. [logging] fix logging init if linking in kudu client

[kudu-CR] [tools] ignore SIGPIPE in kudu CLI

2018-07-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has abandoned this change. ( http://gerrit.cloudera.org:8080/10938 ) Change subject: [tools] ignore SIGPIPE in kudu CLI .. Abandoned Obsoleted by https://gerrit.cloudera.org/#/c/10946/ -- To view, visit

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-07-13 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/10406 ) Change subject: KUDU-2437 Split a tablet into primary key ranges by size .. Patch Set 21: (8 comments)

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-07-13 Thread Yao Xu (Code Review)
Hello Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#20). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] [tools] ignore SIGPIPE in kudu CLI

2018-07-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10938 ) Change subject: [tools] ignore SIGPIPE in kudu CLI .. Patch Set 1: > (1 comment) Yep, sure -- I'll do. But so far I'm sure the

[kudu-CR] spark: Add support for scanRequestTimeout to DefaultSource

2018-07-13 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10725 ) Change subject: spark: Add support for scanRequestTimeout to DefaultSource .. spark: Add support for scanRequestTimeout to

[kudu-CR] spark: Expose socketReadTimeoutMs to Spark connector

2018-07-13 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/10839 ) Change subject: spark: Expose socketReadTimeoutMs to Spark connector .. Patch Set 4: (1 comment)

[kudu-CR] [tools] ignore SIGPIPE in kudu CLI

2018-07-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10938 Change subject: [tools] ignore SIGPIPE in kudu CLI .. [tools] ignore SIGPIPE in kudu CLI Since OpenSSL does not provide much control

[kudu-CR] WIP Add error handling to KuduBackup DONT BUILD KuduBackup will attempt to backup every table. If any one table fails the entire job will fail. All exceptions are logged when they happen.

2018-07-13 Thread Tony Foerster (Code Review)
Tony Foerster has posted comments on this change. ( http://gerrit.cloudera.org:8080/10941 ) Change subject: WIP Add error handling to KuduBackup DONT_BUILD KuduBackup will attempt to backup every table. If any one table fails the entire job will fail. All exceptions are logged when they

[kudu-CR] WIP Add error handling to KuduBackup DONT BUILD KuduBackup will attempt to backup every table. If any one table fails the entire job will fail. All exceptions are logged when they happen.

2018-07-13 Thread Tony Foerster (Code Review)
Tony Foerster has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10941 Change subject: WIP Add error handling to KuduBackup DONT_BUILD KuduBackup will attempt to backup every table. If any one table fails the entire job will fail. All exceptions are logged when they

[kudu-CR] Change yetus dependency to compile in spark modules

2018-07-13 Thread Tony Foerster (Code Review)
Tony Foerster has posted comments on this change. ( http://gerrit.cloudera.org:8080/10927 ) Change subject: Change yetus dependency to compile in spark modules .. Patch Set 2: Code-Review+1 -- To view, visit

[kudu-CR] Change yetus dependency to compile in spark modules

2018-07-13 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10927 ) Change subject: Change yetus dependency to compile in spark modules .. Change yetus dependency to compile in spark modules Due

[kudu-CR] [tools] ignore SIGPIPE in kudu CLI

2018-07-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10938 ) Change subject: [tools] ignore SIGPIPE in kudu CLI .. Patch Set 1: (1 comment)

[kudu-CR](branch-1.6.x) KUDU-2312: Scan predicate application ordering is non-deterministic

2018-07-13 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/10945 ) Change subject: KUDU-2312: Scan predicate application ordering is non-deterministic .. Patch Set 1: This applied cleanly except for

[kudu-CR] spark: Expose socketReadTimeoutMs to Spark connector

2018-07-13 Thread Mike Percy (Code Review)
Hello Dan Burkert, Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10839 to look at the new patch set (#5). Change subject: spark: Expose socketReadTimeoutMs to Spark connector

[kudu-CR] [tools] ignore SIGPIPE in kudu CLI

2018-07-13 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10938 ) Change subject: [tools] ignore SIGPIPE in kudu CLI .. Patch Set 1: (1 comment)

[kudu-CR] [tools] ignore SIGPIPE in kudu CLI

2018-07-13 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10938 ) Change subject: [tools] ignore SIGPIPE in kudu CLI .. Patch Set 1: (1 comment)

[kudu-CR] [tools] ignore SIGPIPE in kudu CLI

2018-07-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10938 ) Change subject: [tools] ignore SIGPIPE in kudu CLI .. Patch Set 1: (1 comment)

[kudu-CR](branch-1.6.x) KUDU-2312: Scan predicate application ordering is non-deterministic

2018-07-13 Thread Dan Burkert (Code Review)
Hello Attila Bukor, Kudu Jenkins, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/10945 to review the following change. Change subject: KUDU-2312: Scan predicate application ordering is non-deterministic