[kudu-CR] KUDU-1821. Noisy warning from catalog manager

2017-01-10 Thread Jun He (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5637 to look at the new patch set (#3). Change subject: KUDU-1821. Noisy warning from catalog manager .. KUDU-1821. Nois

[kudu-CR] KUDU-1821. Noisy warning from catalog manager

2017-01-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1821. Noisy warning from catalog manager .. Patch Set 3: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/5637 To unsubscribe, visit http://ge

[kudu-CR] [kudu-jepsen] Kudu Jepsen tests

2017-01-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [kudu-jepsen] Kudu Jepsen tests .. Patch Set 16: (19 comments) http://gerrit.cloudera.org:8080/#/c/5492/16/java/kudu-jepsen/.gitignore File java/kudu-jepsen/.gi

[kudu-CR] [kudu-jepsen] Kudu Jepsen tests

2017-01-10 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5492 to look at the new patch set (#18). Change subject: [kudu-jepsen] Kudu Jepsen tests .. [kudu-jepsen] Kudu Jepsen te

[kudu-CR] [kudu-jepsen] added Jenkins script

2017-01-10 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5624 to look at the new patch set (#6). Change subject: [kudu-jepsen] added Jenkins script .. [kudu-jepsen] added Jenkin

[kudu-CR] [c++ client] implemented session operations stats

2017-01-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has abandoned this change. Change subject: [c++ client] implemented session operations stats .. Abandoned It seems this is not needed in this form -- Impala client could maintain these metrics itself, if needed.

[kudu-CR] [webserver] small font in the footer

2017-01-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/5670 Change subject: [webserver] small font in the footer .. [webserver] small font in the footer This is to have a proper separation between

[kudu-CR] KUDU-1821. Noisy warning from catalog manager

2017-01-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1821. Noisy warning from catalog manager .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5637 To unsubscribe, visit http://ger

[kudu-CR] KUDU-1821. Noisy warning from catalog manager

2017-01-10 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: KUDU-1821. Noisy warning from catalog manager .. KUDU-1821. Noisy warning from catalog manager This commit changes the LOG if the catalog manager is trying

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

2017-01-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: docs: KUDU-1767. Document possible client op reordering .. Patch Set 4: (8 comments) http://gerrit.cloudera.org:8080/#/c/5464/4//COMMIT_MSG Commit Message: PS4

[kudu-CR] KUDU-1733. Update consistency semantics doc

2017-01-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1733. Update consistency semantics doc .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5605 To unsubscribe, visit http://gerri

[kudu-CR] [webserver] small font in the footer

2017-01-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [webserver] small font in the footer .. Patch Set 1: Does this change the look & feel when styles are enabled? Should we just add a

[kudu-CR] KUDU-1741: Keep MiniCluster::Restart consistent with ExternalMiniCluster::Restart

2017-01-10 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1741: Keep MiniCluster::Restart consistent with ExternalMiniCluster::Restart .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.o

[kudu-CR] [webserver] small font in the footer

2017-01-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [webserver] small font in the footer .. Patch Set 1: Or just set your $KUDU_HOME env var to your kudu checkout directory if it bugs you? -- To view, visit http:

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

2017-01-10 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 4: (2 comments) http://gerrit.cloudera.org:8080/#/c/5464/4/docs/transaction_semantics.

[kudu-CR] KUDU-1733. Update consistency semantics doc

2017-01-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1733. Update consistency semantics doc .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5605 To unsubscribe, visit http://ge

[kudu-CR] KUDU-1733. Update consistency semantics doc

2017-01-10 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 Reviewed-o

[kudu-CR] [webserver] small font in the footer

2017-01-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [webserver] small font in the footer .. Patch Set 1: > Or just set your $KUDU_HOME env var to your kudu checkout directory > if it bugs you? Yeah, that's what I d

[kudu-CR] KUDU-1741: Keep MiniCluster::Restart consistent with ExternalMiniCluster::Restart

2017-01-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1741: Keep MiniCluster::Restart consistent with ExternalMiniCluster::Restart .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/

[kudu-CR] KUDU-1741: Keep MiniCluster::Restart consistent with ExternalMiniCluster::Restart

2017-01-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1741: Keep MiniCluster::Restart consistent with ExternalMiniCluster::Restart .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/

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

2017-01-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/5671 Change subject: [TLS certs management] initial commit .. [TLS certs management] initial commit Added code for TLS certificate management

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

2017-01-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/5672 Change subject: [TLS cert management] TLS cert generation unit tests .. [TLS cert management] TLS cert generation unit tests Change-Id:

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

2017-01-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/5673 Change subject: [TLS cert management] added protobuf interface .. [TLS cert management] added protobuf interface Added protobuf interfac

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

2017-01-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/5674 Change subject: [TLS cert management] security service implementation .. [TLS cert management] security service implementation Added imp

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

2017-01-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/5675 Change subject: [TLS cert management] security service units tests .. [TLS cert management] security service units tests Added unit test

[kudu-CR] WIP: [util/crypto] certificate management (part 1)

2017-01-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has abandoned this change. Change subject: WIP: [util/crypto] certificate management (part 1) .. Abandoned This changelist is split into the 567{1..5}. The idea behind the split is to ease the review process --

[kudu-CR] KUDU-1741: Keep MiniCluster::Restart consistent with ExternalMiniCluster::Restart

2017-01-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. Change subject: KUDU-1741: Keep MiniCluster::Restart consistent with ExternalMiniCluster::Restart .. KUDU-1741: Keep MiniCluster::Restart consistent with ExternalMiniC

[kudu-CR] KUDU-1741: Keep MiniCluster::Restart consistent with ExternalMiniCluster::Restart

2017-01-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1741: Keep MiniCluster::Restart consistent with ExternalMiniCluster::Restart .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:80

[kudu-CR] Doc review for 1.2

2017-01-10 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Ambreen Kazi, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5676 to review the following change. Change subject: Doc review for 1.2 .. Doc review for 1.2 I made

[kudu-CR] [webserver] small font in the footer

2017-01-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has abandoned this change. Change subject: [webserver] small font in the footer .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/5670 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

[kudu-CR] [webserver] small font in the footer

2017-01-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [webserver] small font in the footer .. Patch Set 1: > > Or just set your $KUDU_HOME env var to your kudu checkout > directory > > if it bugs you? > > Yeah,