[kudu-CR] [make site.sh] an option for the script: doxygen

2016-07-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/3766 Change subject: [make_site.sh] an option for the script: doxygen .. [make_site.sh] an option for the script: doxygen By default, do not

[kudu-CR] KUDU-1311 [master] support adding and dropping range partitions

2016-07-22 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1311 [master] support adding and dropping range partitions .. Patch Set 14: (4 comments) Looked only at the doxygen comments in client.h file.

[kudu-CR] [make site.h] added C++ client API documentaion

2016-07-22 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3729 to look at the new patch set (#2). Change subject: [make_site.h] added C++ client API documentaion ..

[kudu-CR] [KuduScanBatch::const iterator] a minor clean-up

2016-08-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/3834 Change subject: [KuduScanBatch::const_iterator] a minor clean-up .. [KuduScanBatch::const_iterator] a minor clean-up More

[kudu-CR] c++ client: remove unnecessary code

2016-08-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: c++ client: remove unnecessary code .. Patch Set 3: Code-Review+1 (2 comments) http://gerrit.cloudera.org:8080/#/c/3809/3/src/kudu/client/client-internal.cc

[kudu-CR] [KuduScanBatch::const iterator] a minor clean-up

2016-08-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [KuduScanBatch::const_iterator] a minor clean-up .. Patch Set 1: (3 comments) Thank you for review. Will post an updated version soon.

[kudu-CR] [doxygen] made DESTDIR applicable for ninja builds

2016-08-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/3825 Change subject: [doxygen] made DESTDIR applicable for ninja builds .. [doxygen] made DESTDIR applicable for ninja builds Moved the

[kudu-CR] KUDU-456 Implement AUTO FLUSH BACKGROUND flush mode

2016-08-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/3952 Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode Implemented

[kudu-CR] [KuduPartialRow::Set{Binary,String}()] copy input data

2016-08-10 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3868 to look at the new patch set (#9). Change subject: [KuduPartialRow::Set{Binary,String}()] copy input data ..

[kudu-CR] C++ client: deprecating KuduPartialRow::SetString()

2016-08-10 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3723 to look at the new patch set (#9). Change subject: C++ client: deprecating KuduPartialRow::SetString()

[kudu-CR] [KuduPartialRow::Set{Binary,String}()] copy input data

2016-08-09 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3868 to look at the new patch set (#6). Change subject: [KuduPartialRow::Set{Binary,String}()] copy input data ..

[kudu-CR] C++ client: deprecating KuduPartialRow::SetString()

2016-08-10 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3723 to look at the new patch set (#11). Change subject: C++ client: deprecating KuduPartialRow::SetString()

[kudu-CR] [KuduPartialRow::Set{Binary,String}()] copy input data

2016-08-10 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3868 to look at the new patch set (#10). Change subject: [KuduPartialRow::Set{Binary,String}()] copy input data ..

[kudu-CR] [util] fixed build on MacOS X

2016-08-03 Thread Alexey Serbin (Code Review)
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3836 to look at the new patch set (#2). Change subject: [util] fixed build on MacOS X .. [util] fixed

[kudu-CR] [C++ client] doxygenized all C++ client API

2016-08-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/3840 Change subject: [C++ client] doxygenized all C++ client API .. [C++ client] doxygenized all C++ client API Doxygenized the rest of

[kudu-CR] KUDU-456 Implement AUTO FLUSH BACKGROUND flush mode

2016-08-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has abandoned this change. Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. Abandoned Abandoned in favor of https://gerrit.cloudera.org/#/c/3952/ The new approach uses RPC messenger thread

[kudu-CR] C++ client: deprecating KuduPartialRow::SetString()

2016-08-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has abandoned this change. Change subject: C++ client: deprecating KuduPartialRow::SetString() .. Abandoned Abandoning this in favor of http://gerrit.cloudera.org:8080/3868 which has been merged already. -- To

[kudu-CR] [util/env posix.cc] fixed build on MacOS X

2016-08-15 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [util/env_posix.cc] fixed build on MacOS X .. Patch Set 1: > oops, my bad! np, and thank you for the instant review! -- To view, visit

[kudu-CR] KUDU-456 Implement AUTO FLUSH BACKGROUND flush mode

2016-08-15 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3952 to look at the new patch set (#5). Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. KUDU-456

[kudu-CR] [util/stopwatch.h] fixed integer overflow

2016-08-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/3960 Change subject: [util/stopwatch.h] fixed integer overflow .. [util/stopwatch.h] fixed integer overflow Fixed integer overflow reported

[kudu-CR] KUDU-456 Implement AUTO FLUSH BACKGROUND flush mode

2016-08-13 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3952 to look at the new patch set (#4). Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. KUDU-456

[kudu-CR] KUDU-456 Implement AUTO FLUSH BACKGROUND flush mode

2016-08-11 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3668 to look at the new patch set (#8). Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. KUDU-456

[kudu-CR] KUDU-456 Implement AUTO FLUSH BACKGROUND flush mode

2016-08-12 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3952 to look at the new patch set (#2). Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. KUDU-456

[kudu-CR] KUDU-564 (part 1): log a 'diff' when tablet config changes

2016-08-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-564 (part 1): log a 'diff' when tablet config changes .. Patch Set 1: Code-Review+1 A tiny nit: is it possible to re-format the commit message to preserve

[kudu-CR] [KuduPartialRow::Set{Binary,String}()] copy input data

2016-08-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [KuduPartialRow::Set{Binary,String}()] copy input data .. Patch Set 10: (1 comment) Thank you for the review, Mike!

[kudu-CR] [C++ client] doxygenized all C++ client API

2016-08-04 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3840 to look at the new patch set (#2). Change subject: [C++ client] doxygenized all C++ client API .. [C++ client]

[kudu-CR] KUDU-456 Implement AUTO FLUSH BACKGROUND flush mode

2016-08-05 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3668 to look at the new patch set (#5). Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. KUDU-456

[kudu-CR] Rename Remote Bootstrap to Tablet Copy (part 1)

2016-08-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Rename Remote Bootstrap to Tablet Copy (part 1) .. Patch Set 2: Code-Review+1 > Build Started http://104.196.14.100/job/kudu-gerrit/2720/ The renaming of files

[kudu-CR] [CMakeLists.txt] dot is mandatory for doxygen

2016-08-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/3855 Change subject: [CMakeLists.txt] dot is mandatory for doxygen .. [CMakeLists.txt] dot is mandatory for doxygen Require dot (graphviz)

[kudu-CR] C++ client: deprecating KuduPartialRow::SetString()

2016-08-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: C++ client: deprecating KuduPartialRow::SetString() .. Patch Set 3: (5 comments) http://gerrit.cloudera.org:8080/#/c/3723/3//COMMIT_MSG Commit Message: Line

[kudu-CR] client/sample.cc: fixed a couple of crashes

2016-08-08 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3685 to look at the new patch set (#3). Change subject: client/sample.cc: fixed a couple of crashes ..

[kudu-CR] [doxygen] made DESTDIR applicable for ninja builds

2016-08-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [doxygen] made DESTDIR applicable for ninja builds .. Patch Set 1: > Looks good provided you've tested it with both make and ninja. Yes, I tested it with GNU

[kudu-CR] C++ client: deprecating KuduPartialRow::SetString()

2016-08-09 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3723 to look at the new patch set (#5). Change subject: C++ client: deprecating KuduPartialRow::SetString()

[kudu-CR] [util/monotime] added handy operators for MonoTime

2016-08-16 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [util/monotime] added handy operators for MonoTime .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3999/1/src/kudu/util/monotime.cc File

[kudu-CR] [util/monotime] added handy operators for MonoTime

2016-08-16 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [util/monotime] added handy operators for MonoTime .. Patch Set 1: > I was going to say that the style guide frowns upon overloading, > but apparently it was

[kudu-CR] [util/monotime] added handy operators for MonoTime

2016-08-16 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [util/monotime] added handy operators for MonoTime .. Patch Set 1: > (1 comment) Great! That's simplifies everything. Then I'll remove those obsolete checks.

[kudu-CR] [util/monotime] added handy operators for MonoTime

2016-08-16 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [util/monotime] added handy operators for MonoTime .. Patch Set 1: > I was going to say that the style guide frowns upon overloading, > but apparently it was

[kudu-CR] KUDU-456 Implement AUTO FLUSH BACKGROUND flush mode

2016-08-15 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3952 to look at the new patch set (#8). Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. KUDU-456

[kudu-CR](gh-pages) Add weekly update for 8/16

2016-08-16 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Add weekly update for 8/16 .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/4008/1/_posts/2016-08-16-weekly-update.md File

[kudu-CR] [util/monotime] added handy operators for MonoTime

2016-08-16 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3999 to look at the new patch set (#3). Change subject: [util/monotime] added handy operators for MonoTime ..

[kudu-CR] tool: better handling for positional arguments

2016-08-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: tool: better handling for positional arguments .. Patch Set 1: > (5 comments) > > There is a Go library which allows to implement rich CLI tools. If >

[kudu-CR] [util/monotime] added handy operators for MonoTime

2016-08-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [util/monotime] added handy operators for MonoTime .. Patch Set 5: (1 comment) Thank you for the review!

[kudu-CR](gh-pages) Add weekly update for 8/16

2016-08-16 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Add weekly update for 8/16 .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4008 To unsubscribe, visit

[kudu-CR] tls socket: support zero-length writes

2017-02-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: tls_socket: support zero-length writes .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5861 To unsubscribe, visit

[kudu-CR] [mini cluster] fix crash in GetLeaderMasterIndex()

2017-02-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [mini_cluster] fix crash in GetLeaderMasterIndex() .. Patch Set 3: Verified+1 Unrelated failure in DEBUG configuration: kudu/tests/test_scantoken.py

[kudu-CR] Allow configuring TlsContext with key wrappers

2017-02-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Allow configuring TlsContext with key wrappers .. Patch Set 4: (6 comments) http://gerrit.cloudera.org:8080/#/c/5845/4/src/kudu/security/tls_context.cc File

[kudu-CR] [security] adjust TLS certificate verification

2017-02-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [security] adjust TLS certificate verification .. Patch Set 3: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8080/#/c/5865/3/src/kudu/rpc/sasl_common.h

[kudu-CR] security: simplify CertSigner interface

2017-02-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: security: simplify CertSigner interface .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5846 To unsubscribe, visit

[kudu-CR] Fix some clang-tidy errors for std::move

2017-02-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Fix some clang-tidy errors for std::move .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5867 To unsubscribe, visit

[kudu-CR] [master] store CA information in the system table

2017-02-01 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5793 to look at the new patch set (#6). Change subject: [master] store CA information in the system table .. [master]

[kudu-CR] client: rename various function calls and classes to ConnectToCluster

2017-02-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: client: rename various function calls and classes to ConnectToCluster .. Patch Set 1: Code-Review+2 (1 comment) LGTM, consider fixing a tiny typo before

[kudu-CR] [mini cluster] fix crash in GetLeaderMasterIndex()

2017-02-01 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5870 to look at the new patch set (#3). Change subject: [mini_cluster] fix crash in GetLeaderMasterIndex() ..

[kudu-CR] [mini cluster] fix crash in GetLeaderMasterIndex()

2017-02-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new patch set (#2). Change subject: [mini_cluster] fix crash in GetLeaderMasterIndex() .. [mini_cluster] fix crash in GetLeaderMasterIndex() If at least one of the mini-masters is shutdown by

[kudu-CR] [master] store CA information in the system table

2017-02-01 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5793 to look at the new patch set (#8). Change subject: [master] store CA information in the system table .. [master]

[kudu-CR] [mini cluster] fix crash in GetLeaderMasterIndex()

2017-02-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/5870 Change subject: [mini_cluster] fix crash in GetLeaderMasterIndex() .. [mini_cluster] fix crash in GetLeaderMasterIndex() If at least

[kudu-CR] WIP: issue authentication tokens and CA certs to clients

2017-02-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: WIP: issue authentication tokens and CA certs to clients .. Patch Set 1: (4 comments)

[kudu-CR] [master] store CA information in the system table

2017-02-02 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5793 to look at the new patch set (#9). Change subject: [master] store CA information in the system table .. [master]

[kudu-CR] [master] store CA information in the system table

2017-02-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [master] store CA information in the system table .. Patch Set 10: (11 comments)

[kudu-CR] [master] store CA information in the system table

2017-02-03 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5793 to look at the new patch set (#11). Change subject: [master] store CA information in the system table .. [master]

[kudu-CR] [master] store CA information in the system table

2017-02-03 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5793 to look at the new patch set (#10). Change subject: [master] store CA information in the system table .. [master]

[kudu-CR] [master] store CA information in the system table

2017-02-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [master] store CA information in the system table .. Patch Set 11: (16 comments) Thank you for review. I spot some extra things myself and decided to update

[kudu-CR] [sys catalog] make visitor's code more generic

2017-02-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/5901 Change subject: [sys_catalog] make visitor's code more generic .. [sys_catalog] make visitor's code more generic Minor refactoring on

[kudu-CR] [sys catalog] make visitor's code more generic

2017-02-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [sys_catalog] make visitor's code more generic .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5901/1/src/kudu/master/sys_catalog.cc File

[kudu-CR] [sys catalog] make visitor's code more generic

2017-02-03 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5901 to look at the new patch set (#3). Change subject: [sys_catalog] make visitor's code more generic .. [sys_catalog]

[kudu-CR] [sys catalog] make visitor's code more generic

2017-02-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [sys_catalog] make visitor's code more generic .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/5901/1/src/kudu/master/sys_catalog.cc File

[kudu-CR] [sys catalog] make visitor's code more generic

2017-02-03 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5901 to look at the new patch set (#2). Change subject: [sys_catalog] make visitor's code more generic .. [sys_catalog]

[kudu-CR] Allow configuring TlsContext with key wrappers

2017-02-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Allow configuring TlsContext with key wrappers .. Patch Set 6: Code-Review+2 (1 comment)

[kudu-CR] [master] store CA information in the system table

2017-02-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [master] store CA information in the system table .. Patch Set 11: (1 comment) http://gerrit.cloudera.org:8080/#/c/5793/11/src/kudu/master/catalog_manager.cc

[kudu-CR] [master] store CA information in the system table

2017-02-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [master] store CA information in the system table .. Patch Set 11: (7 comments) http://gerrit.cloudera.org:8080/#/c/5793/11/src/kudu/master/master.proto File

[kudu-CR] [master] store CA information in the system table

2017-02-03 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5793 to look at the new patch set (#13). Change subject: [master] store CA information in the system table .. [master]

[kudu-CR] master: issue authentication tokens and CA certs to clients

2017-02-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: master: issue authentication tokens and CA certs to clients .. Patch Set 4: (4 comments)

[kudu-CR] [master] store CA information in the system table

2017-02-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [master] store CA information in the system table .. Patch Set 16: Verified+1 I think the test failure in

[kudu-CR] [master] store CA information in the system table

2017-02-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. Change subject: [master] store CA information in the system table .. [master] store CA information in the system table The certificate authority information (private

[kudu-CR] [master] store CA information in the system table

2017-02-03 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5793 to look at the new patch set (#12). Change subject: [master] store CA information in the system table .. [master]

[kudu-CR] [master] store CA information in the system table

2017-02-03 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5793 to look at the new patch set (#15). Change subject: [master] store CA information in the system table .. [master]

[kudu-CR] [master] store CA information in the system table

2017-02-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [master] store CA information in the system table .. Patch Set 14: (1 comment) http://gerrit.cloudera.org:8080/#/c/5793/14/src/kudu/master/catalog_manager.cc

[kudu-CR] [master] store CA information in the system table

2017-02-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [master] store CA information in the system table .. Patch Set 11: (1 comment)

[kudu-CR] [master] store CA information in the system table

2017-02-03 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5793 to look at the new patch set (#14). Change subject: [master] store CA information in the system table .. [master]

[kudu-CR] [master] store CA information in the system table

2017-02-03 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5793 to look at the new patch set (#16). Change subject: [master] store CA information in the system table

[kudu-CR] [master] store CA information in the system table

2017-02-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [master] store CA information in the system table .. Patch Set 15: (1 comment) http://gerrit.cloudera.org:8080/#/c/5793/15/src/kudu/master/catalog_manager.cc

[kudu-CR] KUDU-1864 Thirdparty squeasel build fails on macOS 10.12

2017-02-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/5907 Change subject: KUDU-1864 Thirdparty squeasel build fails on macOS 10.12 .. KUDU-1864 Thirdparty squeasel build fails on macOS 10.12

[kudu-CR] [security] sign/verify data using RSA key pair

2017-01-31 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/5805 to look at the new patch set (#14). Change subject: [security] sign/verify data using RSA key pair ..

[kudu-CR] [security] sign/verify data using RSA key pair

2017-01-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [security] sign/verify data using RSA key pair .. Patch Set 13: (2 comments) http://gerrit.cloudera.org:8080/#/c/5805/13//COMMIT_MSG Commit Message: PS13,

[kudu-CR] [security] do actual token signing/verification

2017-01-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [security] do actual token signing/verification .. Patch Set 11: Verified+1 Unrelated breakage: 19:15:06 CMake Error at CMakeLists.txt:917 (find_package):

[kudu-CR] [security] do actual token signing/verification

2017-01-31 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5812 to look at the new patch set (#11). Change subject: [security] do actual token signing/verification .. [security]

[kudu-CR] tls: hook up internal PKI system to TlsContext

2017-01-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: tls: hook up internal PKI system to TlsContext .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/5808/2/src/kudu/security/tls_context.cc File

[kudu-CR] tls: move setting of verification modes into TlsHandshake

2017-01-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: tls: move setting of verification modes into TlsHandshake .. Patch Set 1: (1 comment)

[kudu-CR] [security] method to extract public part of an RSA key

2017-01-31 Thread Alexey Serbin (Code Review)
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5783 to look at the new patch set (#14). Change subject: [security] method to extract public part of an RSA key

[kudu-CR] [security] method to extract public part of an RSA key

2017-01-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [security] method to extract public part of an RSA key .. Patch Set 13: (2 comments)

[kudu-CR] [security] avoid crashing when importing invalid TSKs

2017-01-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [security] avoid crashing when importing invalid TSKs .. Patch Set 1: Code-Review+2 (1 comment)

[kudu-CR] [security] sign/verify data using RSA key pair

2017-01-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. Change subject: [security] sign/verify data using RSA key pair .. [security] sign/verify data using RSA key pair Added functionality to make a signature of a data

[kudu-CR] [security] avoid crashing when importing invalid TSKs

2017-01-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [security] avoid crashing when importing invalid TSKs .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/5843/1/src/kudu/security/token-test.cc

[kudu-CR] [security] introduced crypto-test

2017-01-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. Change subject: [security] introduced crypto-test .. [security] introduced crypto-test The new crypto-test module is for test scenarios involving common non-CA crypto

[kudu-CR] [security] avoid crashing when importing invalid TSKs

2017-01-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. Change subject: [security] avoid crashing when importing invalid TSKs .. [security] avoid crashing when importing invalid TSKs Kudu typically tries to avoid crashing

[kudu-CR] [security] do actual token signing/verification

2017-01-31 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5812 to look at the new patch set (#10). Change subject: [security] do actual token signing/verification .. [security]

[kudu-CR] [security] do actual token signing/verification

2017-01-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [security] do actual token signing/verification .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5812/2/src/kudu/security/token_signing_key.cc

[kudu-CR] [security] do actual token signing/verification

2017-01-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [security] do actual token signing/verification .. Patch Set 11: (2 comments) http://gerrit.cloudera.org:8080/#/c/5812/11/src/kudu/security/token_signing_key.h

[kudu-CR] [security] do actual token signing/verification

2017-01-31 Thread Alexey Serbin (Code Review)
Hello Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5812 to look at the new patch set (#12). Change subject: [security] do actual token signing/verification .. [security]

[kudu-CR] [security] do actual token signing/verification

2017-01-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [security] do actual token signing/verification .. Patch Set 12: Code-Review+2 Propagating +2 from Todd's review to this version (comment updates). -- To

[kudu-CR] [security] do actual token signing/verification

2017-01-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. Change subject: [security] do actual token signing/verification .. [security] do actual token signing/verification Replaced temporary stub sign/verify calls with calls

  1   2   3   4   5   6   7   8   9   10   >