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

kishore gopalakrishna commented on S4-5:
----------------------------------------

Hi Leo,

I really like this approach hiding the Guice details underneath. 

Few thoughts,

* Feels like there are multiple ways to add PE to streams ( while creating the 
StreamMaker s1,s2,s3) and then am.addPE.to(s2).to(s3).
* will it help separating the two one step to add stream and one step to addPE 
to stream. 

Also it looks like keyFinder is tied to stream  
'am.addStream(EventA.class).withName("My first 
stream.").withKeyFinder("{gender}")' but its actually tied not because one can 
have many keyFinder on the streams.

Does this make sense

s1 = am.addStream("My first stream").ofType(EventA.class)

am.onStream(s1).invoke(PEX.class).usingKeyFinder("{gender}")

Do we really need StreamMaker and PEMaker









                
> Prototype app builder
> ---------------------
>
>                 Key: S4-5
>                 URL: https://issues.apache.org/jira/browse/S4-5
>             Project: Apache S4
>          Issue Type: New Feature
>    Affects Versions: 0.5
>            Reporter: Leo Neumeyer
>            Priority: Minor
>             Fix For: 0.5
>
>
> Extend Guice AbstractModule to create a simple application builder layer. The 
> idea is to evaluate if we can use Guice to do all the wiring including PEs 
> and Streams and Apps.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to