Re: Review Request 50886: config page load takes long time on cluster with large number of config versions

2016-08-10 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50886/#review145334 --- Ship it! Ship It! - Jonathan Hurley On Aug. 9, 2016, 6:45 p.

Re: Review Request 50886: config page load takes long time on cluster with large number of config versions

2016-08-09 Thread Jaimin Jetly
> On Aug. 9, 2016, 6:22 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java, > > lines 3558-3565 > > > > > > This logic will not use

Re: Review Request 50886: config page load takes long time on cluster with large number of config versions

2016-08-09 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50886/ --- (Updated Aug. 9, 2016, 10:45 p.m.) Review request for Ambari, Dmytro Sen, Jonat

Re: Review Request 50886: config page load takes long time on cluster with large number of config versions

2016-08-09 Thread Jaimin Jetly
> On Aug. 9, 2016, 6:22 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java, > > line 21 > > > > > > Could you adjust your import sorti

Re: Review Request 50886: config page load takes long time on cluster with large number of config versions

2016-08-09 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50886/#review145229 --- ambari-server/src/main/java/org/apache/ambari/server/controller/A

Review Request 50886: config page load takes long time on cluster with large number of config versions

2016-08-08 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50886/ --- Review request for Ambari, Dmytro Sen, Jonathan Hurley, Myroslav Papirkovskyy, S