Management optimization: don't send (empty) statistic updates for object

classes that don't have statistics.


git-svn-id: https://svn.apache.org/repos/asf/incubator/qpid/trunk/qpid@724356 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 3 insertions, 6 deletions

diff --git a/cpp/src/qpid/management/ManagementBroker.cpp b/cpp/src/qpid/management/ManagementBroker.cpp
index 7082859948..cc7a2dc4f3 100644
--- a/cpp/src/qpid/management/ManagementBroker.cpp
+++ b/cpp/src/qpid/management/ManagementBroker.cpp
@@ -360,15 +360,13 @@ void ManagementBroker::periodicProcessing (void)
 
     for (ManagementObjectMap::iterator iter = managementObjects.begin ();
          iter != managementObjects.end ();
-         iter++)
-    {
+         iter++) {
         ManagementObject* object = iter->second;
 
         if (object->getConfigChanged() || object->getInstChanged())
             object->setUpdateTime();
 
-        if (object->getConfigChanged() || object->getForcePublish() || object->isDeleted())
-        {
+        if (object->getConfigChanged() || object->getForcePublish() || object->isDeleted()) {
             Buffer msgBuffer (msgChars, BUFSIZE);
             encodeHeader (msgBuffer, 'c');
             object->writeProperties(msgBuffer);
@@ -379,8 +377,7 @@ void ManagementBroker::periodicProcessing (void)
             sendBuffer (msgBuffer, contentSize, mExchange, routingKey);
         }
         
-        if (object->getInstChanged() || object->getForcePublish())
-        {
+        if (object->hasInst() && (object->getInstChanged() || object->getForcePublish())) {
             Buffer msgBuffer (msgChars, BUFSIZE);
             encodeHeader (msgBuffer, 'i');
             object->writeStatistics(msgBuffer);