You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@storm.apache.org by et...@apache.org on 2020/01/31 14:23:08 UTC

[storm] branch master updated: STORM-3570 add config name when failing validation

This is an automated email from the ASF dual-hosted git repository.

ethanli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/storm.git


The following commit(s) were added to refs/heads/master by this push:
     new 4e0ebd4  STORM-3570 add config name when failing validation
     new 7373774  Merge pull request #3200 from agresch/agresch_storm_3570
4e0ebd4 is described below

commit 4e0ebd4cbc23927c846ee31530b2d29dd4f338da
Author: Aaron Gresch <ag...@yahoo-inc.com>
AuthorDate: Wed Jan 29 14:40:29 2020 -0600

    STORM-3570 add config name when failing validation
---
 storm-client/src/jvm/org/apache/storm/validation/ConfigValidation.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/storm-client/src/jvm/org/apache/storm/validation/ConfigValidation.java b/storm-client/src/jvm/org/apache/storm/validation/ConfigValidation.java
index f5db47c..f52df59 100644
--- a/storm-client/src/jvm/org/apache/storm/validation/ConfigValidation.java
+++ b/storm-client/src/jvm/org/apache/storm/validation/ConfigValidation.java
@@ -901,7 +901,7 @@ public class ConfigValidation {
                     LOG.warn("We loosen some constraints here to support topologies of older version running on the current version");
                     validateField(name, className.replace("backtype.storm", "org.apache.storm"));
                 } else {
-                    throw new RuntimeException(e);
+                    throw new RuntimeException("Failed to validate config " + name + " with value " + className, e);
                 }
             }
         }