Re: Review Request 56426: AMBARI-19912 - Manage Config Group host list is not sorted

2017-02-08 Thread Richard Zang
> On Feb. 8, 2017, 7:28 p.m., Jaimin Jetly wrote: > > ambari-web/app/mappers/configs/config_groups_mapper.js, line 71 > > > > > > sorting a list might not be very expensive on client side. But if there > > are

Re: Review Request 56426: AMBARI-19912 - Manage Config Group host list is not sorted

2017-02-08 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56426/#review164761 --- Ship it! Ship It! - Jaimin Jetly On Feb. 8, 2017, 8:18

Re: Review Request 56426: AMBARI-19912 - Manage Config Group host list is not sorted

2017-02-08 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56426/ --- (Updated Feb. 8, 2017, 8:18 p.m.) Review request for Ambari, Jaimin Jetly and

Re: Review Request 56426: AMBARI-19912 - Manage Config Group host list is not sorted

2017-02-08 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56426/#review164745 ---

Review Request 56426: AMBARI-19912 - Manage Config Group host list is not sorted

2017-02-07 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56426/ --- Review request for Ambari and Xi Wang. Bugs: AMBARI-19912