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

On Tue, Feb 4, 2014 at 5:42 PM, Summers Pittman <supittma(a)redhat.com <mailto:supittma@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.reci... > [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/s... > _______________________________________________ > aerogear-dev mailing list > aerogear-dev(a)lists.jboss.org <mailto:aerogear-dev@lists.jboss.org> > https://lists.jboss.org/mailman/listinfo/aerogear-dev _______________________________________________ aerogear-dev mailing list aerogear-dev(a)lists.jboss.org <mailto:aerogear-dev@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(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/aerogear-dev