[ 
https://issues.apache.org/jira/browse/S4-105?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13495219#comment-13495219
 ] 

Daniel Gómez Ferro commented on S4-105:
---------------------------------------

Change looks good and tests pass, please merge!
                
> Incorrect method implementation in org.apache.s4.core.Stream
> ------------------------------------------------------------
>
>                 Key: S4-105
>                 URL: https://issues.apache.org/jira/browse/S4-105
>             Project: Apache S4
>          Issue Type: Bug
>    Affects Versions: 0.5.0
>            Reporter: Xinwu Cheng
>            Assignee: Matthieu Morel
>             Fix For: 0.6
>
>   Original Estimate: 1m
>  Remaining Estimate: 1m
>
> In class org.apache.s4.core.Stream:
>     public Stream<T> setPE(ProcessingElement pe) {
>         app.addStream(this);
>         return this;
>     }
> This method implementation is wrong, should add argument pe into targetPEs 
> list, not addStream() 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to