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

[Matthias Wessendorf]

On Tue, Feb 4, 2014 at 6:10 PM, Summers Pittman <supittma at redhat.com> wrote:

>  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>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.
>

Ah! ok :-)

At least it sounds like a good idea to leverage the pipes internally,
instead of making a AGSyncPipe


>
>
>
>
>> 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
>
>
> _______________________________________________
> aerogear-dev mailing listaerogear-dev at lists.jboss.orghttps://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/319bca42/attachment-0001.html