[kudu-CR] Avoid extra fsyncs of tombstoned tablets during startup

2016-11-04 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: Avoid extra fsyncs of tombstoned tablets during startup .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/4915/3/src/kudu/tablet/tablet_metadata.cc

[kudu-CR] [OpenSSL] require at least 1.0.0 version

2016-11-04 Thread Alexey Serbin (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4937 to look at the new patch set (#3). Change subject: [OpenSSL] require at least 1.0.0 version ..

[kudu-CR] Improve debuggability of the delta/compaction path

2016-11-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Improve debuggability of the delta/compaction path .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/4930/2/src/kudu/tablet/delta_store.cc File

[kudu-CR] Add a macro to LOG and return on a non-OK status

2016-11-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Add a macro to LOG and return on a non-OK status .. Patch Set 2: (5 comments) http://gerrit.cloudera.org:8080/#/c/4927/2//COMMIT_MSG Commit Message: Nit: if

[kudu-CR] [ssl] disable SSL/TLS compression

2016-11-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [ssl] disable SSL/TLS compression .. Patch Set 1: (1 comment) > (1 comment) > > Good catch. I wasn't aware of this. Also just FYI, it looks like > it's

[kudu-CR] [ssl] disable SSL/TLS compression

2016-11-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/4962 Change subject: [ssl] disable SSL/TLS compression .. [ssl] disable SSL/TLS compression As for the best recommended practices for

[kudu-CR] ssl: switch to older APIs for initializing SSL

2016-11-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: ssl: switch to older APIs for initializing SSL .. ssl: switch to older APIs for initializing SSL This enables support for OpenSSL 1.0.0 as found on RHEL

[kudu-CR] [java client] Implement RPC tracing, part 2

2016-11-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] Implement RPC tracing, part 2 .. [java client] Implement RPC tracing, part 2 This patch adds pretty printing for the traces, and

[kudu-CR] ssl: switch to older APIs for initializing SSL

2016-11-04 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: ssl: switch to older APIs for initializing SSL .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4957 To unsubscribe, visit

[kudu-CR] Fix broken GFlags documentation link

2016-11-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Fix broken GFlags documentation link .. Fix broken GFlags documentation link Change-Id: Ib67847245608bfe2b8bd827c34aa27ae051d97a3 Reviewed-on:

[kudu-CR] rpc: move connection header send/receive into negotiation.cc

2016-11-04 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, Sailesh Mukil, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4959 to review the following change. Change subject: rpc: move connection header send/receive into negotiation.cc

[kudu-CR] ssl: switch to older APIs for initializing SSL

2016-11-04 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, Sailesh Mukil, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4957 to look at the new patch set (#2). Change subject: ssl: switch to older APIs for initializing SSL

[kudu-CR] [java client] Redirect KuduExceptions to RowError in KuduSession

2016-11-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Redirect KuduExceptions to RowError in KuduSession .. Patch Set 3: > I'll defer the +2 to Alexey; he's most familiar with the equivalent >

[kudu-CR] [java client] Redirect KuduExceptions to RowError in KuduSession

2016-11-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java client] Redirect KuduExceptions to RowError in KuduSession .. Patch Set 3: Code-Review+1 I'll defer the +2 to Alexey; he's most familiar with the equivalent

[kudu-CR] [java client] Implement RPC tracing, part 2

2016-11-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java client] Implement RPC tracing, part 2 .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4950 To unsubscribe, visit

[kudu-CR] ssl: switch to older APIs for initializing SSL

2016-11-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: ssl: switch to older APIs for initializing SSL .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/4957/1/src/kudu/util/net/ssl_factory.cc File

[kudu-CR] cmake: search for all required Kerberos binaries

2016-11-04 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: cmake: search for all required Kerberos binaries .. cmake: search for all required Kerberos binaries If one or more binaries are not found, CMake will

[kudu-CR] [java client] Implement RPC tracing, part 1

2016-11-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] Implement RPC tracing, part 1 .. [java client] Implement RPC tracing, part 1 First part of this work is adding the tracing objects

[kudu-CR] [rpc] Close socket on non-linux platforms during RPC shutdown

2016-11-04 Thread Dinesh Bhat (Code Review)
Hello Mike Percy, Adar Dembo, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4958 to review the following change. Change subject: [rpc] Close socket on non-linux platforms during RPC shutdown

[kudu-CR] ssl: switch to older APIs for initializing SSL

2016-11-04 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: ssl: switch to older APIs for initializing SSL .. Patch Set 1: (1 comment) After reading the attached discussion, LGTM.

[kudu-CR] ssl: switch to older APIs for initializing SSL

2016-11-04 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, Sailesh Mukil, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4957 to review the following change. Change subject: ssl: switch to older APIs for initializing SSL

[kudu-CR] Fix broken GFlags documentation link

2016-11-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Fix broken GFlags documentation link .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4955 To unsubscribe, visit

[kudu-CR] cmake: search for all required Kerberos binaries

2016-11-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: cmake: search for all required Kerberos binaries .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4954 To unsubscribe, visit

[kudu-CR] [java client] Implement RPC tracing, part 1

2016-11-04 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java client] Implement RPC tracing, part 1 .. Patch Set 9: Code-Review+2 thanks! -- To view, visit http://gerrit.cloudera.org:8080/4781 To unsubscribe, visit

[kudu-CR] Avoid extra fsyncs of tombstoned tablets during startup

2016-11-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Avoid extra fsyncs of tombstoned tablets during startup .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/4915/3/src/kudu/tablet/tablet_metadata.cc

[kudu-CR] cmake: search for all required Kerberos binaries

2016-11-04 Thread Dan Burkert (Code Review)
Hello Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4954 to look at the new patch set (#2). Change subject: cmake: search for all required Kerberos binaries .. cmake: search

[kudu-CR] [java client] Implement RPC tracing, part 1

2016-11-04 Thread Jean-Daniel Cryans (Code Review)
Hello Adar Dembo, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4781 to look at the new patch set (#9). Change subject: [java client] Implement RPC tracing, part 1

[kudu-CR] Fix broken GFlags documentation link

2016-11-04 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has uploaded a new change for review. http://gerrit.cloudera.org:8080/4955 Change subject: Fix broken GFlags documentation link .. Fix broken GFlags documentation link Change-Id:

[kudu-CR] [java client] Redirect KuduExceptions to RowError in KuduSession

2016-11-04 Thread Jean-Daniel Cryans (Code Review)
Hello Adar Dembo, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4949 to look at the new patch set (#3). Change subject: [java client] Redirect KuduExceptions to RowError in KuduSession

[kudu-CR] [java client] Implement RPC tracing, part 2

2016-11-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java client] Implement RPC tracing, part 2 .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4950 To unsubscribe, visit

[kudu-CR] cmake: search for all required Kerberos binaries

2016-11-04 Thread Dan Burkert (Code Review)
Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4954 to review the following change. Change subject: cmake: search for all required Kerberos binaries .. cmake: search for

[kudu-CR] [java client] Implement RPC tracing, part 1

2016-11-04 Thread Jean-Daniel Cryans (Code Review)
Hello Adar Dembo, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4781 to look at the new patch set (#8). Change subject: [java client] Implement RPC tracing, part 1

[kudu-CR] [java client] Implement RPC tracing, part 2

2016-11-04 Thread Jean-Daniel Cryans (Code Review)
Hello Adar Dembo, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4950 to look at the new patch set (#3). Change subject: [java client] Implement RPC tracing, part 2

[kudu-CR] [java client] Implement RPC tracing, part 1

2016-11-04 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java client] Implement RPC tracing, part 1 .. Patch Set 5: (1 comment) sorz

[kudu-CR] Add KuduTable.getFormattedRangePartitions method

2016-11-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add KuduTable.getFormattedRangePartitions method .. Patch Set 2: (8 comments)

[kudu-CR] KUDU-1735. Fix crash when aborting a skipped config change round

2016-11-04 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4916 to look at the new patch set (#3). Change subject: KUDU-1735. Fix crash when aborting a skipped config change round

[kudu-CR] [java client] Implement RPC tracing, part 2

2016-11-04 Thread Jean-Daniel Cryans (Code Review)
Hello Adar Dembo, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4950 to look at the new patch set (#2). Change subject: [java client] Implement RPC tracing, part 2

[kudu-CR] [java client] Redirect KuduExceptions to RowError in KuduSession

2016-11-04 Thread Jean-Daniel Cryans (Code Review)
Hello Adar Dembo, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4949 to look at the new patch set (#2). Change subject: [java client] Redirect KuduExceptions to RowError in KuduSession

[kudu-CR] [java client] Redirect KuduExceptions to RowError in KuduSession

2016-11-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Redirect KuduExceptions to RowError in KuduSession .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/4949/1//COMMIT_MSG

[kudu-CR] [tools] Manual recovery tools (part 1)

2016-11-04 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [tools] Manual recovery tools (part 1) .. Patch Set 10: (2 comments) http://gerrit.cloudera.org:8080/#/c/4834/6/src/kudu/tools/kudu-tool-test.cc File

[kudu-CR] [java client] Implement RPC tracing, part 2

2016-11-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java client] Implement RPC tracing, part 2 .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/4950/1//COMMIT_MSG Commit Message: PS1, Line 10: We

[kudu-CR] [java client] Redirect KuduExceptions to RowError in KuduSession

2016-11-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java client] Redirect KuduExceptions to RowError in KuduSession .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/4949/1//COMMIT_MSG Commit

[kudu-CR] [java client] Implement RPC tracing, part 1

2016-11-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java client] Implement RPC tracing, part 1 .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4781 To unsubscribe, visit

[kudu-CR] installation.adoc: add openssl requirement for macOS

2016-11-04 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: installation.adoc: add openssl requirement for macOS .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/4952/1/docs/installation.adoc File

[kudu-CR] installation.adoc: add openssl requirement for macOS

2016-11-04 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4952 to look at the new patch set (#2). Change subject: installation.adoc: add openssl requirement for macOS ..

[kudu-CR] block manager: better preallocation in log block manager

2016-11-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: block_manager: better preallocation in log block manager .. block_manager: better preallocation in log block manager As has been discussed to

[kudu-CR] block manager: better preallocation in log block manager

2016-11-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: block_manager: better preallocation in log block manager .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4848 To

[kudu-CR] [java client] Cleanup "Connection reset" message in TabletClient

2016-11-04 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: [java client] Cleanup "Connection reset" message in TabletClient .. [java client] Cleanup "Connection reset" message in TabletClient It was adding a

[kudu-CR] [java client] Cleanup "Connection reset" message in TabletClient

2016-11-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java client] Cleanup "Connection reset" message in TabletClient .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4947 To

[kudu-CR] installation.adoc: add openssl requirement for macOS

2016-11-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: installation.adoc: add openssl requirement for macOS .. Patch Set 1: (3 comments) We also need to doc the openssl requirement on Linux. Do you want to do that

[kudu-CR] installation.adoc: add openssl requirement for macOS

2016-11-04 Thread Dan Burkert (Code Review)
Hello Adar Dembo, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4952 to review the following change. Change subject: installation.adoc: add openssl requirement for macOS ..

[kudu-CR] [tools] Manual recovery tools (part 1)

2016-11-04 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [tools] Manual recovery tools (part 1) .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4834/1/src/kudu/tools/tool_action_local_replica.cc File

[kudu-CR] [tools] Manual recovery tools (part 1)

2016-11-04 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [tools] Manual recovery tools (part 1) .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/4834/6/src/kudu/tools/kudu-tool-test.cc File

[kudu-CR] [tools] Manual recovery tools (part 1)

2016-11-04 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [tools] Manual recovery tools (part 1) .. Patch Set 10: (2 comments) > Would you mind also adding a test that makes sure that attempting > to copy over an

[kudu-CR] [tools] Manual recovery tools (part 1)

2016-11-04 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [tools] Manual recovery tools (part 1) .. Patch Set 10: (2 comments) > (2 comments) > > Would you mind also adding a test that makes sure that attempting >

[kudu-CR] Reject CREATE TABLE ops with even replication factor

2016-11-04 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: Reject CREATE TABLE ops with even replication factor .. Patch Set 2: Working on the fix.. -- To view, visit http://gerrit.cloudera.org:8080/4945 To unsubscribe,

[kudu-CR] [java client] Implement RPC tracing, part 2

2016-11-04 Thread Jean-Daniel Cryans (Code Review)
Hello Adar Dembo, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4950 to review the following change. Change subject: [java client] Implement RPC tracing, part 2 .. [java

[kudu-CR] [java client] Redirect KuduExceptions to RowError in KuduSession

2016-11-04 Thread Jean-Daniel Cryans (Code Review)
Hello Adar Dembo, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4949 to review the following change. Change subject: [java client] Redirect KuduExceptions to RowError in KuduSession

[kudu-CR] [java client] Cleanup "Connection reset" message in TabletClient

2016-11-04 Thread Jean-Daniel Cryans (Code Review)
Hello Adar Dembo, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4947 to review the following change. Change subject: [java client] Cleanup "Connection reset" message in TabletClient

[kudu-CR] [tools] Manual recovery tools (part 1)

2016-11-04 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [tools] Manual recovery tools (part 1) .. Patch Set 10: > maybe we should always force Actually, what I think we should do is attempt to send the actual

[kudu-CR] Reject CREATE TABLE ops with even replication factor

2016-11-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Reject CREATE TABLE ops with even replication factor .. Patch Set 2: > Build Failed > > http://104.196.14.100/job/kudu-gerrit/4280/ : FAILURE Looks

[kudu-CR] block manager: disk space checking everywhere

2016-11-04 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: block_manager: disk space checking everywhere .. Patch Set 5: Apparently needs a rebase, please feel free to carry my +2 through -- To view, visit

[kudu-CR] block manager: various changes to disk space reservation checking

2016-11-04 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: block_manager: various changes to disk space reservation checking .. block_manager: various changes to disk space reservation checking I was trying to

[kudu-CR] block manager: cosmetic changes to the LBM

2016-11-04 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: block_manager: cosmetic changes to the LBM .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4847 To unsubscribe, visit

[kudu-CR] block manager: various changes to disk space reservation checking

2016-11-04 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: block_manager: various changes to disk space reservation checking .. Patch Set 4: (3 comments)

[kudu-CR] KUDU-100 enable the ability to use RLE for the int64 type

2016-11-04 Thread Haijie Hong (Code Review)
Haijie Hong has uploaded a new change for review. http://gerrit.cloudera.org:8080/4946 Change subject: KUDU-100 enable the ability to use RLE for the int64 type .. KUDU-100 enable the ability to use RLE for the int64 type As

[kudu-CR] Reject CREATE TABLE ops with even replication factor

2016-11-04 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new patch set (#2). Change subject: Reject CREATE TABLE ops with even replication factor .. Reject CREATE TABLE ops with even replication factor Reject table creation with even replication factor, and

[kudu-CR] Reject CREATE TABLE ops with even replication factor

2016-11-04 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/4945 Change subject: Reject CREATE TABLE ops with even replication factor .. Reject CREATE TABLE ops with even replication factor Reject table