----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23127/#review46858 -----------------------------------------------------------
Ship it! ambari-server/src/main/java/org/apache/ambari/server/api/query/QueryImpl.java <https://reviews.apache.org/r/23127/#comment82424> Just so I understand... you are checking if the predicate contains any properties that haven't been set yet by the resource provider, right? It's easier than checking all of the property providers. Good idea. I don't think that subResourcePropertiesInpredicate is a good name. The set will contain any properties of the top level resource that aren't supported by the resource provider. It is really a set of unsupported property ids. ambari-server/src/main/java/org/apache/ambari/server/api/query/QueryImpl.java <https://reviews.apache.org/r/23127/#comment82425> The comment is no longer correct. Maybe ... // If there is a page request and the predicate does not contain properties that need to be set ambari-server/src/main/java/org/apache/ambari/server/api/query/QueryImpl.java <https://reviews.apache.org/r/23127/#comment82426> Can we remove dead code? ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterControllerImpl.java <https://reviews.apache.org/r/23127/#comment82427> This was just to help organize the class methods. I think it helps visualize what a class is used for when you can see all of its public methods together. I use the headings to try to keep things organized. I'm not sure why you want to remove it but whatever... - Tom Beerbower On June 27, 2014, 1:40 p.m., Dmytro Sen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/23127/ > ----------------------------------------------------------- > > (Updated June 27, 2014, 1:40 p.m.) > > > Review request for Ambari, Sid Wagle and Tom Beerbower. > > > Bugs: AMBARI-6306 > https://issues.apache.org/jira/browse/AMBARI-6306 > > > Repository: ambari > > > Description > ------- > > Slowness of filtering operations on hosts, opening a JIRA to track the > performance aspect of it. > In general, filtering operations available on the Hosts page (such as > filtering by host_status, hostname, ip, etc., are slow and takes more than 40 > seconds on a 2k-node cluster). > > After the patch response time is 1-3 seconds, if there is no subresources in > predicate > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/api/query/QueryImpl.java > ec8a3d4 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterControllerImpl.java > 3f8f317 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StackDefinedPropertyProvider.java > 51c7565 > > ambari-server/src/main/java/org/apache/ambari/server/controller/spi/ClusterController.java > d7d916e > > Diff: https://reviews.apache.org/r/23127/diff/ > > > Testing > ------- > > Fixing > > > Thanks, > > Dmytro Sen > >
