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

[Matthias Wessendorf]

On Tue, Feb 4, 2014 at 5:42 PM, Summers Pittman <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


> 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
> > https://lists.jboss.org/mailman/listinfo/aerogear-dev
>
> _______________________________________________
> aerogear-dev mailing list
> 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/aerogear-dev/attachments/20140204/f001d0ec/attachment-0001.html