Re: [kudu-CR] [TLS certs management] initial commit

2017-01-12 Thread Alexey Serbin
Right; that version were posted accidentally -- I actually only posted the update for the next patch (unit tests), not going to post the update for the first one because it was not ready yet. But gerrit updated this one as it sensed the base has been changed as well. Please hold on reviewing

[kudu-CR] KUDU-1771. Java client "connection refused" errors logged as "connection reset"

2017-01-12 Thread Jun He (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5680 to look at the new patch set (#2). Change subject: KUDU-1771. Java client "connection refused" errors logged as "connection reset"

[kudu-CR] KUDU-1771. Java client "connection refused" errors logged as "connection reset"

2017-01-12 Thread Jun He (Code Review)
Jun He has posted comments on this change. Change subject: KUDU-1771. Java client "connection refused" errors logged as "connection reset" .. Patch Set 1: (2 comments)

[kudu-CR] KUDU-1595 and KUDU-1642: IS NOT NULL and IS NULL predicates

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1595 and KUDU-1642: IS NOT NULL and IS NULL predicates .. Patch Set 2: (12 comments) Looks pretty good. Dan should probably take a look too.

[kudu-CR] [TLS certs management] initial commit

2017-01-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [TLS certs management] initial commit .. Patch Set 2: crypto_engine and crypto_common are no longer needed, right? -- To view, visit

[kudu-CR] Remove Schema#toString from RowResult#toString

2017-01-12 Thread Kousuke Saruta (Code Review)
Kousuke Saruta has posted comments on this change. Change subject: Remove Schema#toString from RowResult#toString .. Patch Set 1: No problem. Thank you for taking a look at this :) -- To view, visit

[kudu-CR] Make transaction semantics docs more consistent

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has abandoned this change. Change subject: Make transaction semantics docs more consistent .. Abandoned Incorporated these changes into http://gerrit.cloudera.org:8080/5605 -- To view, visit

[kudu-CR] Remove Schema#toString from RowResult#toString

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Remove Schema#toString from RowResult#toString .. Patch Set 1: Dan/JD - what do you guys think about this? (sorry Kousuke for the slow response to this! lost

[kudu-CR] [TLS cert management] added protobuf interface

2017-01-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [TLS cert management] added protobuf interface .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/5673/1/src/kudu/security/security.proto File

[kudu-CR] [TLS cert management] TLS cert generation unit tests

2017-01-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [TLS cert management] TLS cert generation unit tests .. Patch Set 1: (1 comment)

[kudu-CR](branch-1.2.x) Update version to non-snapshot 1.2

2017-01-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Update version to non-snapshot 1.2 .. Update version to non-snapshot 1.2 Change-Id: I9730ba902956d03c0baf244d51555ff339da45c9 Reviewed-on:

[kudu-CR](branch-1.2.x) Update version to non-snapshot 1.2

2017-01-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Update version to non-snapshot 1.2 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5700 To unsubscribe, visit

[kudu-CR] [TLS certs management] initial commit

2017-01-12 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5671 to look at the new patch set (#2). Change subject: [TLS certs management] initial commit .. [TLS certs management]

[kudu-CR] [TLS cert management] TLS cert generation unit tests

2017-01-12 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5672 to look at the new patch set (#2). Change subject: [TLS cert management] TLS cert generation unit tests .. [TLS

[kudu-CR] Add openssl util

2017-01-12 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: Add openssl_util .. Add openssl_util Also migrates the existing ssl_factory static initialization to use the new openssl_util version. Change-Id:

[kudu-CR] Move SSLFactory and SSLSocket to the security module

2017-01-12 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: Move SSLFactory and SSLSocket to the security module .. Move SSLFactory and SSLSocket to the security module Soon these classes will rely on the security

[kudu-CR] [TLS cert management] security service units tests

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [TLS cert management] security service units tests .. Patch Set 1: (4 comments)

[kudu-CR] [TLS cert management] security service implementation

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [TLS cert management] security service implementation .. Patch Set 1: (9 comments)

[kudu-CR] [TLS cert management] TLS cert generation unit tests

2017-01-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [TLS cert management] TLS cert generation unit tests .. Patch Set 1: (7 comments)

[kudu-CR] Move SSLFactory and SSLSocket to the security module

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Move SSLFactory and SSLSocket to the security module .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/5703/5/src/kudu/security/x509_check_host.h

[kudu-CR] [TLS cert management] added protobuf interface

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [TLS cert management] added protobuf interface .. Patch Set 1: (10 comments) http://gerrit.cloudera.org:8080/#/c/5673/1/src/kudu/security/security.proto File

[kudu-CR] Move SSLFactory and SSLSocket to the security module

2017-01-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Move SSLFactory and SSLSocket to the security module .. Patch Set 6: Code-Review+2 (1 comment)

[kudu-CR] Add openssl util

2017-01-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Add openssl_util .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5704 To unsubscribe, visit

[kudu-CR] webserver: improve SSL certificate handling

2017-01-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: webserver: improve SSL certificate handling .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/5015/5/src/kudu/server/webserver_options.cc File

[kudu-CR] [TLS cert management] TLS cert generation unit tests

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [TLS cert management] TLS cert generation unit tests .. Patch Set 1: (7 comments)

[kudu-CR] rpc: allow using encrypted private key for SSL

2017-01-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: rpc: allow using encrypted private key for SSL .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5692/1/src/kudu/rpc/messenger.cc File

[kudu-CR] Move SSLFactory and SSLSocket to the security module

2017-01-12 Thread Dan Burkert (Code Review)
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5703 to look at the new patch set (#6). Change subject: Move SSLFactory and SSLSocket to the security module

[kudu-CR] Move SSLFactory and SSLSocket to the security module

2017-01-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Move SSLFactory and SSLSocket to the security module .. Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/5703/5//COMMIT_MSG Commit Message: PS5,

[kudu-CR] webserver: improve SSL certificate handling

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: webserver: improve SSL certificate handling .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/5015/5/src/kudu/server/webserver_options.cc File

[kudu-CR] Move SSLFactory and SSLSocket to the security module

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Move SSLFactory and SSLSocket to the security module .. Patch Set 5: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/5703 To unsubscribe, visit

[kudu-CR] Add openssl util

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add openssl_util .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5704 To unsubscribe, visit

[kudu-CR] Add openssl util

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add openssl_util .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/5704/6/src/kudu/security/CMakeLists.txt File src/kudu/security/CMakeLists.txt:

[kudu-CR] Move SSLFactory and SSLSocket to the security module

2017-01-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Move SSLFactory and SSLSocket to the security module .. Patch Set 5: (4 comments) http://gerrit.cloudera.org:8080/#/c/5703/5//COMMIT_MSG Commit Message: PS5,

[kudu-CR] Add openssl util

2017-01-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add openssl_util .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/5704/6/src/kudu/security/CMakeLists.txt File src/kudu/security/CMakeLists.txt:

[kudu-CR] Add openssl util

2017-01-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add openssl_util .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/c/5704/6/src/kudu/security/tls_common.cc File src/kudu/security/tls_common.cc:

[kudu-CR] Move SSLFactory and SSLSocket to the security module

2017-01-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Move SSLFactory and SSLSocket to the security module .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/5703/4/src/kudu/security/x509_check_host.cc

[kudu-CR] Add openssl util

2017-01-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Add openssl_util .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/c/5704/6/src/kudu/security/CMakeLists.txt File

[kudu-CR] Move SSLFactory and SSLSocket to the security module

2017-01-12 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5703 to look at the new patch set (#5). Change subject: Move SSLFactory and SSLSocket to the security module .. Move

[kudu-CR] Add openssl util

2017-01-12 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5704 to look at the new patch set (#7). Change subject: Add openssl_util .. Add openssl_util Also migrates the

[kudu-CR] Add tls common module

2017-01-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add tls_common module .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/c/5704/6/src/kudu/security/CMakeLists.txt File

[kudu-CR] webserver: improve SSL certificate handling

2017-01-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: webserver: improve SSL certificate handling .. Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/5015/5/src/kudu/server/webserver_options.cc File

[kudu-CR](branch-1.2.x) docs: update quickstart to a new dataset

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: docs: update quickstart to a new dataset .. Patch Set 1: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5705 To unsubscribe, visit

[kudu-CR](branch-1.2.x) docs: update quickstart to a new dataset

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: docs: update quickstart to a new dataset .. docs: update quickstart to a new dataset The old dataset we were using seems to have changed formats and no

[kudu-CR](branch-1.2.x) docs: update quickstart to a new dataset

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/5705 Change subject: docs: update quickstart to a new dataset .. docs: update quickstart to a new dataset The old dataset we were using seems

[kudu-CR] docs: update quickstart to a new dataset

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: docs: update quickstart to a new dataset .. docs: update quickstart to a new dataset The old dataset we were using seems to have changed formats and no

[kudu-CR] Add tls common module

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add tls_common module .. Patch Set 6: (5 comments) http://gerrit.cloudera.org:8080/#/c/5704/6/src/kudu/security/CMakeLists.txt File

[kudu-CR] Move SSLFactory and SSLSocket to the security module

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Move SSLFactory and SSLSocket to the security module .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/5703/4/src/kudu/security/x509_check_host.cc

[kudu-CR] [TLS certs management] initial commit

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [TLS certs management] initial commit .. Patch Set 1: (32 comments) http://gerrit.cloudera.org:8080/#/c/5671/1/src/kudu/security/crypto/cert_management.cc File

[kudu-CR] Move SSLFactory and SSLSocket to the security module

2017-01-12 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5703 to look at the new patch set (#4). Change subject: Move SSLFactory and SSLSocket to the security module .. Move

[kudu-CR] Add tls common module

2017-01-12 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5704 to look at the new patch set (#6). Change subject: Add tls_common module .. Add tls_common module tls_common

[kudu-CR] Add tls common module

2017-01-12 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5704 to look at the new patch set (#5). Change subject: Add tls_common module .. Add tls_common module tls_common

[kudu-CR] Move SSLFactory and SSLSocket to the security module

2017-01-12 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5703 to look at the new patch set (#3). Change subject: Move SSLFactory and SSLSocket to the security module .. Move

[kudu-CR] Move SSLFactory and SSLSocket to the security module

2017-01-12 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5703 to look at the new patch set (#2). Change subject: Move SSLFactory and SSLSocket to the security module .. Move

[kudu-CR] Add tls common module

2017-01-12 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5704 to look at the new patch set (#4). Change subject: Add tls_common module .. Add tls_common module tls_common

[kudu-CR] Add tls common module

2017-01-12 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5704 to look at the new patch set (#3). Change subject: Add tls_common module .. Add tls_common module tls_common

[kudu-CR] Add tls common module

2017-01-12 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded a new patch set (#2). Change subject: Add tls_common module .. Add tls_common module tls_common includes common methods for working with the TLS portions of OpenSSL. Change-Id:

[kudu-CR] Move SSLFactory and SSLSocket to the security module

2017-01-12 Thread Dan Burkert (Code Review)
Hello Todd Lipcon, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5703 to review the following change. Change subject: Move SSLFactory and SSLSocket to the security module ..

[kudu-CR] Add tls common module

2017-01-12 Thread Dan Burkert (Code Review)
Hello Todd Lipcon, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5704 to review the following change. Change subject: Add tls_common module .. Add tls_common module

[kudu-CR] docs: update quickstart to a new dataset

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: docs: update quickstart to a new dataset .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5701/2/docs/quickstart.adoc File docs/quickstart.adoc:

[kudu-CR] webserver: improve SSL certificate handling

2017-01-12 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5015 to look at the new patch set (#5). Change subject: webserver: improve SSL certificate handling .. webserver:

[kudu-CR] webserver: improve SSL certificate handling

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: webserver: improve SSL certificate handling .. Patch Set 4: (6 comments) http://gerrit.cloudera.org:8080/#/c/5015/3/src/kudu/server/webserver.cc File

[kudu-CR] docs: update quickstart to a new dataset

2017-01-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: docs: update quickstart to a new dataset .. Patch Set 2: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/5701/2/docs/quickstart.adoc File

[kudu-CR] docs: update quickstart to a new dataset

2017-01-12 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5701 to look at the new patch set (#2). Change subject: docs: update quickstart to a new dataset .. docs: update

[kudu-CR] docs: update quickstart to a new dataset

2017-01-12 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5701 to review the following change. Change subject: docs: update quickstart to a new dataset .. docs: update

[kudu-CR] rpc: allow using encrypted private key for SSL

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: rpc: allow using encrypted private key for SSL .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5692/1/src/kudu/rpc/messenger.cc File

[kudu-CR] rpc: allow using encrypted private key for SSL

2017-01-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: rpc: allow using encrypted private key for SSL .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5692/1/src/kudu/rpc/messenger.cc File

[kudu-CR] rpc: allow using encrypted private key for SSL

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: rpc: allow using encrypted private key for SSL .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5692/1/src/kudu/rpc/messenger.cc File

[kudu-CR] rpc: allow using encrypted private key for SSL

2017-01-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: rpc: allow using encrypted private key for SSL .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5692/1/src/kudu/rpc/messenger.cc File

[kudu-CR](branch-1.2.x) Update version to non-snapshot 1.2

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/5700 Change subject: Update version to non-snapshot 1.2 .. Update version to non-snapshot 1.2 Change-Id:

[kudu-CR] docs: KUDU-1767. Document possible client op reordering

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: docs: KUDU-1767. Document possible client op reordering .. docs: KUDU-1767. Document possible client op reordering This patch documents the possibility

[kudu-CR] docs: KUDU-1767. Document possible client op reordering

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: docs: KUDU-1767. Document possible client op reordering .. Patch Set 7: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5464 To unsubscribe, visit

[kudu-CR](branch-1.2.x) docs: KUDU-1767. Document possible client op reordering

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: docs: KUDU-1767. Document possible client op reordering .. docs: KUDU-1767. Document possible client op reordering This patch documents the possibility

[kudu-CR](branch-1.2.x) docs: KUDU-1767. Document possible client op reordering

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: docs: KUDU-1767. Document possible client op reordering .. Patch Set 1: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5699 To

[kudu-CR] docs: KUDU-1767. Document possible client op reordering

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: docs: KUDU-1767. Document possible client op reordering .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5464 To unsubscribe,

[kudu-CR](branch-1.2.x) docs: KUDU-1767. Document possible client op reordering

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/5699 Change subject: docs: KUDU-1767. Document possible client op reordering .. docs: KUDU-1767. Document possible client op reordering This

[kudu-CR](branch-1.2.x) Initial draft of release notes and doc updates for 1.2

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Initial draft of release notes and doc updates for 1.2 .. Initial draft of release notes and doc updates for 1.2 Change-Id:

[kudu-CR](branch-1.2.x) Initial draft of release notes and doc updates for 1.2

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Initial draft of release notes and doc updates for 1.2 .. Patch Set 1: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5698 To

[kudu-CR](branch-1.2.x) Initial draft of release notes and doc updates for 1.2

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/5698 Change subject: Initial draft of release notes and doc updates for 1.2 .. Initial draft of release notes and doc updates for 1.2

[kudu-CR](branch-1.2.x) Restructure release notes in preparation for 1.2 release

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Restructure release notes in preparation for 1.2 release .. Restructure release notes in preparation for 1.2 release * Moved the 1.1 release notes to the

[kudu-CR](branch-1.2.x) Restructure release notes in preparation for 1.2 release

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Restructure release notes in preparation for 1.2 release .. Patch Set 1: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5697 To

[kudu-CR](branch-1.2.x) Restructure release notes in preparation for 1.2 release

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/5697 Change subject: Restructure release notes in preparation for 1.2 release .. Restructure release notes in preparation for 1.2 release *

[kudu-CR](branch-1.2.x) KUDU-1733. Update consistency semantics doc

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1733. Update consistency semantics doc .. KUDU-1733. Update consistency semantics doc Change-Id: I5a59315e70531c8904470c026030f4cc2107ca5b

[kudu-CR](branch-1.2.x) KUDU-1733. Update consistency semantics doc

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1733. Update consistency semantics doc .. Patch Set 1: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5694 To unsubscribe, visit

[kudu-CR](branch-1.2.x) KUDU-1733. Update consistency semantics doc

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/5694 Change subject: KUDU-1733. Update consistency semantics doc .. KUDU-1733. Update consistency semantics doc Change-Id:

[kudu-CR] [TLS certs management] initial commit

2017-01-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [TLS certs management] initial commit .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5671/1/src/kudu/security/crypto/cert_management.h File

[kudu-CR](branch-1.2.x) KUDU-1822. Update docs to build the documentation in ubuntu

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1822. Update docs to build the documentation in ubuntu .. KUDU-1822. Update docs to build the documentation in ubuntu Include two packages, gem and

[kudu-CR](branch-1.2.x) KUDU-1822. Update docs to build the documentation in ubuntu

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1822. Update docs to build the documentation in ubuntu .. Patch Set 1: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5696 To

[kudu-CR](branch-1.2.x) KUDU-1822. Update docs to build the documentation in ubuntu

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/5696 Change subject: KUDU-1822. Update docs to build the documentation in ubuntu .. KUDU-1822. Update docs to build the documentation in ubuntu

[kudu-CR](branch-1.2.x) docs: update location of devtoolset-3 repo rpm

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: docs: update location of devtoolset-3 repo rpm .. docs: update location of devtoolset-3 repo rpm The old location is now returning a 404 error. Despite

[kudu-CR](branch-1.2.x) docs: update location of devtoolset-3 repo rpm

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/5695 Change subject: docs: update location of devtoolset-3 repo rpm .. docs: update location of devtoolset-3 repo rpm The old location is now

[kudu-CR](branch-1.2.x) docs: update location of devtoolset-3 repo rpm

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: docs: update location of devtoolset-3 repo rpm .. Patch Set 1: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5695 To unsubscribe,

[kudu-CR] docs: KUDU-1767. Document possible client op reordering

2017-01-12 Thread Mike Percy (Code Review)
Hello Todd Lipcon, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5464 to look at the new patch set (#7). Change subject: docs: KUDU-1767. Document possible client op reordering

[kudu-CR] docs: KUDU-1767. Document possible client op reordering

2017-01-12 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: docs: KUDU-1767. Document possible client op reordering .. Patch Set 6: (1 comment)

[kudu-CR] docs: KUDU-1767. Document possible client op reordering

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: docs: KUDU-1767. Document possible client op reordering .. Patch Set 6: Code-Review+1 lgtm modulo JD's point -- To view, visit

[kudu-CR](branch-1.2.x) Doc review for 1.2

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Doc review for 1.2 .. Patch Set 1: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5693 To unsubscribe, visit

[kudu-CR](branch-1.2.x) Doc review for 1.2

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Doc review for 1.2 .. Doc review for 1.2 I made a pass through all of the documentation and made improvements as necessary for 1.2: * Specify that the

[kudu-CR](branch-1.2.x) Doc review for 1.2

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/5693 Change subject: Doc review for 1.2 .. Doc review for 1.2 I made a pass through all of the documentation and made improvements as

[kudu-CR] Doc review for 1.2

2017-01-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Doc review for 1.2 .. Doc review for 1.2 I made a pass through all of the documentation and made improvements as necessary for 1.2: * Specify that the

[kudu-CR] docs: KUDU-1767. Document possible client op reordering

2017-01-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: docs: KUDU-1767. Document possible client op reordering .. Patch Set 6: (1 comment)

[kudu-CR] webserver: improve SSL certificate handling

2017-01-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: webserver: improve SSL certificate handling .. Patch Set 4: (7 comments) http://gerrit.cloudera.org:8080/#/c/5015/3/src/kudu/server/webserver.cc File

  1   2   >