Add Ice.Warn.Executor

zeroc-ice · Feb 5, 2025 · 0c8894a · 0c8894a
1 parent d0533d0
commit 0c8894a
Show file tree

Hide file tree

Showing 6 changed files with 11 additions and 25 deletions.
diff --git a/config/PropertyNames.xml b/config/PropertyNames.xml
@@ -156,6 +156,7 @@
         <property name="Warn.Datagrams" languages="cpp,csharp,java" default="0" />
         <property name="Warn.Dispatch" languages="all" default="1" />
         <property name="Warn.Endpoints" languages="all" default="1" />
+        <property name="Warn.Executor" languages="cpp,csharp,java" default="1" />
         <property name="Warn.UnusedProperties" languages="all" default="0" />
         <property name="CacheMessageBuffers" languages="csharp,java" default="2" />
     </section>

diff --git a/cpp/src/Ice/PropertyNames.cpp b/cpp/src/Ice/PropertyNames.cpp
@@ -193,6 +193,7 @@ const Property IcePropsData[] =
     Property{"Warn.Datagrams", "0", false, false, nullptr},
     Property{"Warn.Dispatch", "1", false, false, nullptr},
     Property{"Warn.Endpoints", "1", false, false, nullptr},
+    Property{"Warn.Executor", "1", false, false, nullptr},
     Property{"Warn.UnusedProperties", "0", false, false, nullptr}
 };
 
@@ -202,7 +203,7 @@ const PropertyArray PropertyNames::IceProps
     .prefixOnly=false,
     .isOptIn=false,
     .properties=IcePropsData,
-    .length=84
+    .length=85
 };
 
 const Property IceMXPropsData[] =

diff --git a/csharp/src/Ice/Internal/PropertiesAdminI.cs b/csharp/src/Ice/Internal/PropertiesAdminI.cs
@@ -166,32 +166,14 @@ public override void
                 // Copy callbacks to allow callbacks to update callbacks
                 foreach (var callback in new List<Ice.PropertiesAdminUpdateCallback>(_deprecatedUpdateCallbacks))
                 {
-                    try
-                    {
-                        callback.updated(changes);
-                    }
-                    catch (System.Exception ex)
-                    {
-                        if (_properties.getIcePropertyAsInt("Ice.Warn.Dispatch") > 1)
-                        {
-                            _logger.warning("properties admin update callback raised unexpected exception:\n" + ex);
-                        }
-                    }
+                    // The callback should not throw any exception.
+                    callback.updated(changes);
                 }
                 // Copy callbacks to allow callbacks to update callbacks
                 foreach (var callback in new List<System.Action<Dictionary<string, string>>>(_updateCallbacks))
                 {
-                    try
-                    {
-                        callback(changes);
-                    }
-                    catch (System.Exception ex)
-                    {
-                        if (_properties.getIcePropertyAsInt("Ice.Warn.Dispatch") > 1)
-                        {
-                            _logger.warning("properties admin update callback raised unexpected exception:\n" + ex);
-                        }
-                    }
+                    // The callback should not throw any exception.
+                    callback(changes);
                 }
             }
         }

diff --git a/csharp/src/Ice/Internal/PropertyNames.cs b/csharp/src/Ice/Internal/PropertyNames.cs
@@ -158,6 +158,7 @@ internal sealed class PropertyNames
             new(pattern: @"Warn.Datagrams", usesRegex: false, defaultValue: "0", deprecated: false, propertyArray: null),
             new(pattern: @"Warn.Dispatch", usesRegex: false, defaultValue: "1", deprecated: false, propertyArray: null),
             new(pattern: @"Warn.Endpoints", usesRegex: false, defaultValue: "1", deprecated: false, propertyArray: null),
+            new(pattern: @"Warn.Executor", usesRegex: false, defaultValue: "1", deprecated: false, propertyArray: null),
             new(pattern: @"Warn.UnusedProperties", usesRegex: false, defaultValue: "0", deprecated: false, propertyArray: null),
             new(pattern: @"CacheMessageBuffers", usesRegex: false, defaultValue: "2", deprecated: false, propertyArray: null)
         ]);

diff --git a/csharp/src/Ice/Internal/ThreadPool.cs b/csharp/src/Ice/Internal/ThreadPool.cs
@@ -333,9 +333,9 @@ public void executeFromThisThread(System.Action call, Ice.Connection connection)
             }
             catch (System.Exception ex)
             {
-                if (_instance.initializationData().properties.getIcePropertyAsInt("Ice.Warn.Dispatch") > 0)
+                if (_instance.initializationData().properties.getIcePropertyAsInt("Ice.Warn.Executor") > 0)
                 {
-                    _instance.initializationData().logger.warning("dispatch exception:\n" + ex);
+                    _instance.initializationData().logger.warning($"executor exception:\n{ex}");
                 }
             }
         }

diff --git a/java/src/com.zeroc.ice/src/main/java/com/zeroc/Ice/PropertyNames.java b/java/src/com.zeroc.ice/src/main/java/com/zeroc/Ice/PropertyNames.java
@@ -162,6 +162,7 @@ final class PropertyNames
             new Property("Warn.Datagrams", false, "0", false, null),
             new Property("Warn.Dispatch", false, "1", false, null),
             new Property("Warn.Endpoints", false, "1", false, null),
+            new Property("Warn.Executor", false, "1", false, null),
             new Property("Warn.UnusedProperties", false, "0", false, null),
             new Property("CacheMessageBuffers", false, "2", false, null)
         });