Re: Review Request 45914: Kerberos: Allow multiple KDC hosts to be set while enabling Kerberos

2016-04-11 Thread Jonathan Hurley
> On April 11, 2016, 8:25 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog240.java, > > lines 1148-1161 > > > > > > Would it be better to just try

Re: Review Request 45914: Kerberos: Allow multiple KDC hosts to be set while enabling Kerberos

2016-04-11 Thread Robert Levas
> On April 11, 2016, 8:25 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog240.java, > > lines 1148-1161 > > > > > > Would it be better to just try

Re: Review Request 45914: Kerberos: Allow multiple KDC hosts to be set while enabling Kerberos

2016-04-11 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45914/#review128097 --- Ship it! Just a comment on the upgrade. Also, are there any

Re: Review Request 45914: Kerberos: Allow multiple KDC hosts to be set while enabling Kerberos

2016-04-11 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45914/#review128091 --- Ship it! Ship It! - Sandor Magyari On April 8, 2016, 9:20

Re: Review Request 45914: Kerberos: Allow multiple KDC hosts to be set while enabling Kerberos

2016-04-08 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45914/#review127793 --- Ship it! Ship It! Maybe you should add some more tests with

Review Request 45914: Kerberos: Allow multiple KDC hosts to be set while enabling Kerberos

2016-04-07 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45914/ --- Review request for Ambari, Jaimin Jetly, Jonathan Hurley, Oliver Szabo, and