> On March 27, 2017, 6:26 p.m., Alejandro Fernandez wrote: > > ambari-agent/src/main/python/ambari_agent/ClusterConfigurationCache.py > > Lines 62 (patched) > > <https://reviews.apache.org/r/57910/diff/6/?file=1676678#file1676678line62> > > > > Inconsistent return value since not all code paths return a value.
This function was implemented long time. It is from alerts. It returns none if case if config is not found. And is checked for alter in single every alert. I only moved it. This is a big change to do that. If think this needs to addressed as a refactoring can you please create another jira. - Andrew ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57910/#review170205 ----------------------------------------------------------- On March 27, 2017, 10:32 a.m., Andrew Onischuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57910/ > ----------------------------------------------------------- > > (Updated March 27, 2017, 10:32 a.m.) > > > Review request for Ambari, Myroslav Papirkovskyy, Sumit Mohanty, and Sid > Wagle. > > > Bugs: AMBARI-20559 > https://issues.apache.org/jira/browse/AMBARI-20559 > > > Repository: ambari > > > Description > ------- > > The class should be: > 1\. Thread-safe > 2\. Support caching for multiple clusters > 3\. Write cache to json file for debug purposes. > > In next jiras the class with be derived by ClusterConfigurationCache, > ClusterTopologyCache and ClusterComponentsStatusCache. > > We have already a caching class for alerts. The jira should also rewrite it to > use the base class. > > > Diffs > ----- > > ambari-agent/src/main/python/ambari_agent/ClusterCache.py PRE-CREATION > ambari-agent/src/main/python/ambari_agent/ClusterConfiguration.py 72b87be > ambari-agent/src/main/python/ambari_agent/ClusterConfigurationCache.py > PRE-CREATION > ambari-agent/src/main/python/ambari_agent/Controller.py 78b5c0c > ambari-agent/src/main/python/ambari_agent/FileCache.py d0c8bdb > ambari-agent/src/main/python/ambari_agent/alerts/base_alert.py 7f3b2a5 > ambari-agent/src/test/python/ambari_agent/TestAlerts.py 64479a2 > ambari-agent/src/test/python/ambari_agent/TestClusterConfigurationCache.py > a418f6d > > > Diff: https://reviews.apache.org/r/57910/diff/6/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Andrew Onischuk > >
