[ 
https://issues.apache.org/jira/browse/STORM-2791?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Daniel Dara updated STORM-2791:
-------------------------------
    Description: 
The class 
[FixedTupleSpout|https://github.com/apache/storm/blob/64e29f365c9b5d3e15b33f33ab64e200345333e4/storm-client/src/jvm/org/apache/storm/testing/FixedTupleSpout.java#L34]
 is useful for emitting test tuples in end-to-end topology tests. Unfortunately 
it appears to only support a single output field, which greatly limits what 
topologies you can test with.

In PR 2391, I propose adding a new constructor to FixedTupleSpout to allow 
specifying more than one field.

I also decided to deprecate the existing constructor which accepts a String due 
to the fact that it will cause an ambiguity compilation error if the calling 
code is passing in `null`, so it should probably be entirely removed at some 
point.

  was:
The class 
[FixedTupleSpout|https://github.com/apache/storm/blob/64e29f365c9b5d3e15b33f33ab64e200345333e4/storm-client/src/jvm/org/apache/storm/testing/FixedTupleSpout.java#L34]
 is useful for emitting test tuples in end-to-end topology tests. Unfortunately 
it appears to only support a single output field, which greatly limits what 
topologies you can test with.

I propose adding a new constructor to FixedTupleSpout to allow specifying more 
than one field.

PR is forthcoming.


> Add support for multiple output fields to FixedTupleSpout
> ---------------------------------------------------------
>
>                 Key: STORM-2791
>                 URL: https://issues.apache.org/jira/browse/STORM-2791
>             Project: Apache Storm
>          Issue Type: Improvement
>          Components: storm-core
>            Reporter: Daniel Dara
>            Priority: Minor
>              Labels: pull-request-available
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> The class 
> [FixedTupleSpout|https://github.com/apache/storm/blob/64e29f365c9b5d3e15b33f33ab64e200345333e4/storm-client/src/jvm/org/apache/storm/testing/FixedTupleSpout.java#L34]
>  is useful for emitting test tuples in end-to-end topology tests. 
> Unfortunately it appears to only support a single output field, which greatly 
> limits what topologies you can test with.
> In PR 2391, I propose adding a new constructor to FixedTupleSpout to allow 
> specifying more than one field.
> I also decided to deprecate the existing constructor which accepts a String 
> due to the fact that it will cause an ambiguity compilation error if the 
> calling code is passing in `null`, so it should probably be entirely removed 
> at some point.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to