----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43538/#review119076 -----------------------------------------------------------
Jayush, there is no reason to take in account config groups when checking selected configs from clusterconfgimapping. Looks like for config groups we have separate logic. Myroslav knows more about that, i've talked to him and he approved this solution. - Vitalyi Brodetskyi On Лют. 12, 2016, 7:56 після полудня, Vitalyi Brodetskyi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43538/ > ----------------------------------------------------------- > > (Updated Лют. 12, 2016, 7:56 після полудня) > > > Review request for Ambari, Jayush Luniya and Myroslav Papirkovskyy. > > > Bugs: AMBARI-15034 > https://issues.apache.org/jira/browse/AMBARI-15034 > > > Repository: ambari > > > Description > ------- > > In clusterconfigmapping table we shouldn't have more than one entries for a > config_type with selected=1 > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/checks/CheckDatabaseHelper.java > a078c8a > > Diff: https://reviews.apache.org/r/43538/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Vitalyi Brodetskyi > >
