[
https://issues.apache.org/jira/browse/STORM-1044?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14900108#comment-14900108
]
ASF GitHub Bot commented on STORM-1044:
---------------------------------------
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/740#discussion_r39936239
--- Diff: storm-core/test/clj/backtype/storm/nimbus_test.clj ---
@@ -846,16 +846,6 @@
NIMBUS-SLOTS-PER-TOPOLOGY 8}]
(letlocals
(bind topology (thrift/mk-topology
- {"1" (thrift/mk-spout-spec (TestPlannerSpout.
true) :parallelism-hint 0 :conf {TOPOLOGY-TASKS 1})}
--- End diff --
Because thrift/mk-topology actually throws IllegalArgumentException, not
submit-local-topology.
We should wrap bind statement to check IllegalArgumentException.
> Setting dop to zero does not raise an error
> -------------------------------------------
>
> Key: STORM-1044
> URL: https://issues.apache.org/jira/browse/STORM-1044
> Project: Apache Storm
> Issue Type: Bug
> Reporter: Matthias J. Sax
> Assignee: Matthias J. Sax
> Priority: Minor
>
> While I did some testing (with automatic topology plugging code) I set the
> dop of all spouts and bolts to zero. I submitted the topology to
> {{LocalCluster}} and did not get any error.
> It took me a while to figure out that the wrongly specified dop were the
> reason for an empty result of the test. From a user point of view, it would
> be nice to raise an exception when the {{parallelism_hint}} is smaller than 1.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
