Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/2759#discussion_r201930088
--- Diff: storm-client/src/jvm/org/apache/storm/utils/Utils.java ---
@@ -781,7 +781,7 @@ private static SerializationDelegate
getSerializationDelegate(Map<String, Object
delegate = (SerializationDelegate) delegateClass.newInstance();
} catch (ClassNotFoundException | InstantiationException |
IllegalAccessException e) {
LOG.error("Failed to construct serialization delegate, falling
back to default", e);
--- End diff --
+1 I would be also OK to not failing back and crash to let users know they
set the class incorrectly.
---
