You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@storm.apache.org by sr...@apache.org on 2019/04/26 14:32:59 UTC

[storm] branch 1.1.x-branch updated: [STORM-3384] set-log-level command throws wrong exception if the topology is not running

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

srdo pushed a commit to branch 1.1.x-branch
in repository https://gitbox.apache.org/repos/asf/storm.git


The following commit(s) were added to refs/heads/1.1.x-branch by this push:
     new 3f783c8  [STORM-3384] set-log-level command throws wrong exception if the topology is not running
     new 164fda9  Merge branch 'STORM-3384-1.1.x' of https://github.com/Ethanlm/storm into asfgit-1.1.x-branch
3f783c8 is described below

commit 3f783c8c82906e4688b8d5d19b7b0b3013eb2cda
Author: Ethan Li <et...@gmail.com>
AuthorDate: Thu Apr 25 09:27:18 2019 -0500

    [STORM-3384] set-log-level command throws wrong exception if the topology is not running
---
 storm-core/src/clj/org/apache/storm/command/set_log_level.clj | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/storm-core/src/clj/org/apache/storm/command/set_log_level.clj b/storm-core/src/clj/org/apache/storm/command/set_log_level.clj
index 7e1c3c5..27dc4c5 100644
--- a/storm-core/src/clj/org/apache/storm/command/set_log_level.clj
+++ b/storm-core/src/clj/org/apache/storm/command/set_log_level.clj
@@ -28,7 +28,7 @@
         topology (first (filter (fn [topo] (= name (.get_name topo))) topologies))]
     (if topology 
       (.get_id topology)
-      (throw (.IllegalArgumentException (str name " is not a running topology"))))))
+      (throw (IllegalArgumentException. (str name " is not a running topology"))))))
 
 (defn- parse-named-log-levels [action]
   "Parses [logger name]=[level string]:[optional timeout],[logger name2]...