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 53a277e  [MINOR] change config validation to warn
     new 3cd0d23  Merge pull request #3250 from 
RuiLi8080/MINOR-Config-Validation
53a277e is described below

commit 53a277ec79d0b289e4f2bef5e94e289132342f9d
Author: Rui Li <[email protected]>
AuthorDate: Thu Apr 16 14:28:51 2020 -0500

    [MINOR] change config validation to warn
---
 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 4bf7432..7d8fcec 100644
--- a/storm-client/src/jvm/org/apache/storm/validation/ConfigValidation.java
+++ b/storm-client/src/jvm/org/apache/storm/validation/ConfigValidation.java
@@ -1017,7 +1017,7 @@ public class ConfigValidation {
                 //To support topologies of older version to run, we might have 
to loose the constraints so that
                 //the configs of older version can pass the validation.
                 if (className.startsWith("backtype.storm")) {
-                    LOG.error("ClassNotFoundException: {}", className);
+                    LOG.warn("ClassNotFoundException: {}", className);
                     LOG.warn("Replace backtype.storm with org.apache.storm and 
try to validate again");
                     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"));

Reply via email to