[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 KerberosOnAndOff/ExternalMiniClusterTest.TestBasicOper

[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 ke

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

2017-02-03 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [master] store CA information in the system table .. Patch Set 16: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5793 To unsubscribe, visit http

[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 Fi

[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 Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [master] store CA information in the system table .. Patch Set 15: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5793 To unsubscribe, visit http

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

2017-02-03 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [master] store CA information in the system table .. Patch Set 15: Code-Review+2 Looks good to me, leaving open for Todd though. -- To view, visit http://gerrit.c

[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 Fi

[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] s

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

2017-02-03 Thread Adar Dembo (Code Review)
Adar Dembo 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 File

[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/master_cert_authority

[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] s

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

2017-02-03 Thread Todd Lipcon (Code Review)
Todd Lipcon 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/master_cert_authority.h

[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 sr

[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] s

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

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

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

2017-02-03 Thread Todd Lipcon (Code Review)
Todd Lipcon 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 src/

[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] s

[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 Fi

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

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

[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 t

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

2017-02-03 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [master] store CA information in the system table .. Patch Set 11: (14 comments) PS11 showed up as I was reviewing PS10, so some of my comments may no longer be r

[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] s

[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) http://gerrit.cloudera.org:8080/#/c/5793/9/src/kudu/integration-tests/CMakeList

[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] s

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

2017-02-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [master] store CA information in the system table .. Patch Set 9: (20 comments) I didn't review the changes to master_cert_authority since I have no context for t

[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] st

[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] st

[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 (#7). Change subject: [master] store CA information in the system table .. [master] st

[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] st