[aerogear-dev] [sync] To pipe or not to pipe?

[Summers Pittman]

On 02/04/2014 11:52 AM, Matthias Wessendorf wrote:
>
>
>
> On Tue, Feb 4, 2014 at 5:42 PM, Summers Pittman <supittma at redhat.com 
> <mailto:supittma at redhat.com>> wrote:
>
>     On 02/04/2014 03:04 AM, Corinne Krych wrote:
>     > Hello all,
>     >
>     > I started doing demo app Buddies and Hobbies [1] (same idea than
>     luke's one [2]). But instead of usin gcomplete separate chanel I
>     wrapped Pipe within SyncPipe [3]. Questions:
>     > 1. looking at Luke's work I didn't see the usage of pipe at all.
>     > 2. do we want to reuse Pipe protocol/interface for sync Pipe or
>     use a different one?
>     > I rather use the pipe underneath as we don't want to reimplement
>     Paging, upload etc...
>     > wdyt guys?
>     I don't think extending Pipe is the best idea. However, I feel
>     that any
>     synchronizer should use pipes under the hood.
>
>
> +1
>
>
>     Think composition over inheritance. Also overloading the read and save
>     methods will get rather rough when we also have to check for conflicts
>     and handle responses from the server.
>
>
> +1
>
>
>     A SyncPipe could be the implementation the sync system works in iOS, 
>
>
> not sure I get this
Sorry was a merge error on my part when I was writing the email.

I meant that there isn't anything wrong with making a sync pipe for 
making implementing sync on iOS easier.  I just don't think it should be 
part of the platform.

>     but
>     I don't think it should be part of the spec/protocol.
>
>
>     >
>     > With our actual server [4], we need adjust to serve REST
>     endpoint. Ugly harcoded it (for now) to serve my purpose.
>     >
>     > ++
>     > Corinne
>     > [1]
>     https://github.com/corinnekrych/aerogear-ios-cookbook/tree/sync.demo.recipe/Hobbies
>     > [2] https://github.com/lholmquist/ag-js-ds-poc
>     > [3] https://github.com/corinnekrych/aerogear-sync-ios
>     > [4]
>     https://github.com/corinnekrych/aerogear-sync-server/blob/master/server/src/main/java/org/jboss/aerogear/sync/rest/RestChannelHandler.java#L49
>     > _______________________________________________
>     > aerogear-dev mailing list
>     > aerogear-dev at lists.jboss.org <mailto:aerogear-dev at lists.jboss.org>
>     > https://lists.jboss.org/mailman/listinfo/aerogear-dev
>
>     _______________________________________________
>     aerogear-dev mailing list
>     aerogear-dev at lists.jboss.org <mailto:aerogear-dev at lists.jboss.org>
>     https://lists.jboss.org/mailman/listinfo/aerogear-dev
>
>
>
>
> -- 
> Matthias Wessendorf
>
> blog: http://matthiaswessendorf.wordpress.com/
> sessions: http://www.slideshare.net/mwessendorf
> twitter: http://twitter.com/mwessendorf
>
>
> _______________________________________________
> aerogear-dev mailing list
> aerogear-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/aerogear-dev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/aerogear-dev/attachments/20140204/8de020e4/attachment.html