Github user srdo commented on a diff in the pull request:
https://github.com/apache/storm/pull/2759#discussion_r201999052
--- 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 --
Thanks, will change it to throw instead.
---
