Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/2752#discussion_r204259528
--- Diff: storm-core/src/jvm/org/apache/storm/command/GetErrors.java ---
@@ -25,6 +25,11 @@
import org.json.simple.JSONValue;
public class GetErrors {
+ /**
+ * Only get errors for a topology.
+ * @param args Used to accept the topology name.
+ * @throws Exception Ignored.
--- End diff --
rewrite this to `@throws Exception on errors` on consistency with
BasicDrpcClient, or just remove `throws Exception`.
---
