----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54888/#review160871 -----------------------------------------------------------
Fix it, then Ship it! Ship It! contrib/views/capacity-scheduler/src/main/resources/ui/app/serializers.js (line 100) <https://reviews.apache.org/r/54888/#comment232122> Remove the commented code. - Gaurav Nagar On Jan. 6, 2017, 10 a.m., Akhil PB wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54888/ > ----------------------------------------------------------- > > (Updated Jan. 6, 2017, 10 a.m.) > > > Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Pallav > Kulshreshtha, and Sreenath Somarajapuram. > > > Bugs: AMBARI-19251 > https://issues.apache.org/jira/browse/AMBARI-19251 > > > Repository: ambari > > > Description > ------- > > Ambari yarn queue manager accepts only integer capacity values, not decimal > capacities. > This patch enables to accept decimal capacities. > > > Diffs > ----- > > > contrib/views/capacity-scheduler/src/main/resources/ui/app/components/capacityInput.js > 8f0246a > contrib/views/capacity-scheduler/src/main/resources/ui/app/serializers.js > ee7bdd4 > > contrib/views/capacity-scheduler/src/main/resources/ui/app/styles/application.less > e24bf4e > > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/capacityEditForm.hbs > 78aca04 > > Diff: https://reviews.apache.org/r/54888/diff/ > > > Testing > ------- > > Manual unit testing > > > Thanks, > > Akhil PB > >
