* Start recording initial complete avatar inventory retrieval failures from the region server

* In theory, this should be a somewhat useless statistic since the user server will already have tried to use the inventory service to retrieve the avatar's skeleton.  If this 
fails, login is halted completely.
* Nonetheless I'm recording it anyway just to see whether it happens (yes, I'm too lazy to scan the logs...)

1 files changed, 23 insertions, 1 deletions

diff --git a/OpenSim/Framework/Statistics/SimExtraStatsCollector.cs b/OpenSim/Framework/Statistics/SimExtraStatsCollector.cs
index 2beb3a0..3716f9b 100644
--- a/OpenSim/Framework/Statistics/SimExtraStatsCollector.cs
+++ b/OpenSim/Framework/Statistics/SimExtraStatsCollector.cs
@@ -43,12 +43,21 @@ namespace OpenSim.Framework.Statistics
         private long assetCacheMemoryUsage;
         private long textureCacheMemoryUsage;
         
+        private long inventoryServiceRetrievalFailures;
+        
         public long AssetsInCache { get { return assetsInCache; } }
         public long TexturesInCache { get { return texturesInCache; } }
         public long AssetCacheMemoryUsage { get { return assetCacheMemoryUsage; } }
         public long TextureCacheMemoryUsage { get { return textureCacheMemoryUsage; } }
         
         /// <summary>
+        /// Number of known failures to retrieve avatar inventory from the inventory service.  This does not
+        /// cover situations where the inventory service accepts the request but never returns any data, since
+        /// we do not yet timeout this situation.
+        /// </summary>
+        public long InventoryServiceRetrievalFailures { get { return inventoryServiceRetrievalFailures; } }
+        
+        /// <summary>
         /// Retain a dictionary of all packet queues stats reporters
         /// </summary>
         private IDictionary<LLUUID, PacketQueueStatsCollector> packetQueueStatsCollectors
@@ -70,6 +79,11 @@ namespace OpenSim.Framework.Statistics
             }
         }  
         
+        public void AddInventoryServiceRetrievalFailure()
+        {
+            inventoryServiceRetrievalFailures++;
+        }
+        
         /// <summary>
         /// Register as a packet queue stats provider
         /// </summary>
@@ -110,7 +124,15 @@ namespace OpenSim.Framework.Statistics
 Texture cache contains {2,6} textures using {3,10:0.000}K" + Environment.NewLine,
                     AssetsInCache, AssetCacheMemoryUsage / 1024.0, 
                     TexturesInCache, TextureCacheMemoryUsage / 1024.0));
-
+            
+            sb.Append(Environment.NewLine);
+            sb.Append("INVENTORY STATISTICS");
+            sb.Append(Environment.NewLine);
+            sb.Append(
+                string.Format(
+                    "Initial inventory caching failures: {0}" + Environment.NewLine,
+                    InventoryServiceRetrievalFailures));
+                                    
             sb.Append(Environment.NewLine);
             sb.Append("PACKET QUEUE STATISTICS");
             sb.Append(Environment.NewLine);