[kudu-CR] Remove unused code for checking PLAIN authentication

2016-10-27 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4874 to review the following change. Change subject: Remove unused code for checking PLAIN authentication ..

[kudu-CR] Remove unused code for checking PLAIN authentication

2016-10-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Remove unused code for checking PLAIN authentication .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/4874/1/src/kudu/rpc/rpc-test-base.h File src

[kudu-CR] Remove unused code for checking PLAIN authentication

2016-10-28 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4874 to look at the new patch set (#2). Change subject: Remove unused code for checking PLAIN authentication .. Remove u

[kudu-CR] Remove unused code for checking PLAIN authentication

2016-10-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Remove unused code for checking PLAIN authentication .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4874/2/src/kudu/rpc/rpc-test-base.h File sr

[kudu-CR] Remove unused code for checking PLAIN authentication

2016-10-28 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4874 to look at the new patch set (#3). Change subject: Remove unused code for checking PLAIN authentication .. Remove u

[kudu-CR] Remove unused code for checking PLAIN authentication

2016-10-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Remove unused code for checking PLAIN authentication .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4874 To unsubscribe, visit

[kudu-CR] Remove unused code for checking PLAIN authentication

2016-10-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Remove unused code for checking PLAIN authentication .. Remove unused code for checking PLAIN authentication This removes the 'AuthStore' abstraction whic