-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58800/
-----------------------------------------------------------

Review request for Ambari, Alejandro Fernandez, Myroslav Papirkovskyy, and Sid 
Wagle.


Bugs: AMBARI-20869
    https://issues.apache.org/jira/browse/AMBARI-20869


Repository: ambari


Description
-------

Changes we discussed with Myroslav Papyrkovskyy to format data in more suitable
for server way.

  * use cluster id instead of name
  * remove some dictionary keys in events to make handing them simpler
  * don't use component name as key in topology dictionary to be ready for 
multiInstances work.


Diffs
-----

  ambari-agent/src/main/python/ambari_agent/ClusterCache.py d5e724c 
  ambari-agent/src/main/python/ambari_agent/ClusterConfigurationCache.py 
03e9645 
  ambari-agent/src/main/python/ambari_agent/ComponentStatusExecutor.py 70cd82a 
  ambari-agent/src/main/python/ambari_agent/Constants.py 5ea916c 
  ambari-agent/src/main/python/ambari_agent/HeartbeatThread.py 6b68f27 
  ambari-agent/src/main/python/ambari_agent/Utils.py 456b2b3 
  ambari-agent/src/test/python/ambari_agent/TestAgentStompResponses.py 7ea0646 
  
ambari-agent/src/test/python/ambari_agent/dummy_files/stomp/components_status_report.json
 c1ce406 
  
ambari-agent/src/test/python/ambari_agent/dummy_files/stomp/configurations_update.json
 41cef61 
  
ambari-agent/src/test/python/ambari_agent/dummy_files/stomp/execution_commands.json
 1eeeb7c 
  
ambari-agent/src/test/python/ambari_agent/dummy_files/stomp/metadata_after_registration.json
 8ccd8e8 
  
ambari-agent/src/test/python/ambari_agent/dummy_files/stomp/metadata_update.json
 96b929b 
  
ambari-agent/src/test/python/ambari_agent/dummy_files/stomp/registration_request.json
 64077b3 
  
ambari-agent/src/test/python/ambari_agent/dummy_files/stomp/registration_response.json
 f744410 
  
ambari-agent/src/test/python/ambari_agent/dummy_files/stomp/topology_update.json
 291249c 


Diff: https://reviews.apache.org/r/58800/diff/1/


Testing
-------

mvn clean test


Thanks,

Andrew Onischuk

Reply via email to