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

Review request for Ambari and Dmitro Lisnichenko.


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


Repository: ambari


Description
-------

As of now, both in python and at puppet status commands are created per-
component. All commands for host are created at once in a loop at
org.apache.ambari.server.agent.HeartbeatMonitor#generateStatusCommands . All
the difference between status commands is a component name and path to script.
So we may reduce number and summary size of commands by sending an entire list
of components in a single status command.  
I'm attaching a dump of a single heartbeat with status commands.

This should also include SECURITY_STATUS merging, which will help in terms of
and similar resolution


Diffs
-----

  ambari-agent/src/main/python/ambari_agent/ActionQueue.py 4ecb822 
  ambari-agent/src/test/python/ambari_agent/TestActionQueue.py 034dba3 
  
ambari-server/src/main/java/org/apache/ambari/server/agent/HeartbeatMonitor.java
 4fe219c 
  ambari-server/src/main/java/org/apache/ambari/server/agent/StatusCommand.java 
6e08ef0 
  
ambari-server/src/test/java/org/apache/ambari/server/agent/TestHeartbeatMonitor.java
 26bf0b9 

Diff: https://reviews.apache.org/r/36542/diff/


Testing
-------

mvn clean test


Thanks,

Andrew Onischuk

Reply via email to