----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64215/#review192453 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java Line 1454 (original), 1453 (patched) <https://reviews.apache.org/r/64215/#comment270605> Do we still need this getCluster() and the null check? - Attila Magyar On Dec. 1, 2017, 8:49 a.m., Sandor Molnar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64215/ > ----------------------------------------------------------- > > (Updated Dec. 1, 2017, 8:49 a.m.) > > > Review request for Ambari, Attila Magyar, Eugene Chekanskiy, and Robert Levas. > > > Bugs: AMBARI-22560 > https://issues.apache.org/jira/browse/AMBARI-22560 > > > Repository: ambari > > > Description > ------- > > Remove hack to set KDC admin credential via the API to set session attribute > via the Cluster resource. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java > 455814aa65 > > ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerImplTest.java > b774436d28 > > > Diff: https://reviews.apache.org/r/64215/diff/1/ > > > Testing > ------- > > Tested on vagrant cluster > > > Thanks, > > Sandor Molnar > >
