> On March 27, 2017, 6:26 p.m., Alejandro Fernandez wrote: > > ambari-agent/src/main/python/ambari_agent/ClusterConfigurationCache.py > > Lines 36 (patched) > > <https://reviews.apache.org/r/57910/diff/6/?file=1676678#file1676678line36> > > > > Please add a description for all of the params in the new files
done in another jira patch > On March 27, 2017, 6:26 p.m., Alejandro Fernandez wrote: > > ambari-agent/src/main/python/ambari_agent/Controller.py > > Lines 236 (patched) > > <https://reviews.apache.org/r/57910/diff/6/?file=1676679#file1676679line236> > > > > Add doc. done in another jira patch - 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 > >
