<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 01/15/2013 06:17 AM, Sebastien Blanc
wrote:<br>
</div>
<blockquote
cite="mid:CAD_dpu3Pg8EH2PnjmNwU3w1FNoYQM17C=PiUCp-5OMXg4yFZvw@mail.gmail.com"
type="cite">
<div dir="ltr">
<div class="gmail_extra">Hi,</div>
<div class="gmail_extra">Some aditional remarks on Summers gist
:</div>
<div class="gmail_extra"><br>
</div>
<div class="gmail_extra">Instead of : </div>
<div class="gmail_extra">
<span
style="line-height:22px;font-size:14px;font-family:Helvetica,arial,freesans,clean,sans-serif">Pipe.readWithFilter(ReadFilter)</span></div>
<div class="gmail_extra"><span
style="line-height:22px;font-size:14px;font-family:Helvetica,arial,freesans,clean,sans-serif">Couldn't
we just overload the read() method ? i.e :</span><br>
</div>
<div class="gmail_extra"><font color="#000000" face="Helvetica,
arial, freesans, clean, sans-serif"><span
style="font-size:14px;line-height:22px">read(ReadFilter
filter)</span></font></div>
<div class="gmail_extra">
<font color="#000000" face="Helvetica, arial, freesans, clean,
sans-serif"><span style="font-size:14px;line-height:22px"><br>
</span></font></div>
</div>
</blockquote>
<font face="Helvetica, arial, freesans, clean, sans-serif">We can
depending on how it interacts with the other implementations of
the library.</font><br>
<blockquote
cite="mid:CAD_dpu3Pg8EH2PnjmNwU3w1FNoYQM17C=PiUCp-5OMXg4yFZvw@mail.gmail.com"
type="cite">
<div dir="ltr">
<div class="gmail_extra"><font color="#000000" face="Helvetica,
arial, freesans, clean, sans-serif"><span
style="font-size:14px;line-height:22px">About the where
property in ReadFilter class :</span></font></div>
<div class="gmail_extra"><font color="#000000" face="Helvetica,
arial, freesans, clean, sans-serif"><span
style="font-size:14px;line-height:22px">Do we really want
to mix paging and querying ? <br>
</span></font></div>
</div>
</blockquote>
<font face="Helvetica, arial, freesans, clean, sans-serif">We do and
we don't. The request itself requires both of these properties so
it makes technical sense to </font>pass them in a single
wrapper. I agree it makes more logical sense to keep them as two
separate object which you can pass independently.<br>
<blockquote
cite="mid:CAD_dpu3Pg8EH2PnjmNwU3w1FNoYQM17C=PiUCp-5OMXg4yFZvw@mail.gmail.com"
type="cite">
<div dir="ltr">
<div class="gmail_extra"><font color="#000000" face="Helvetica,
arial, freesans, clean, sans-serif"><span
style="font-size:14px;line-height:22px">About </span></font><span
style="background-color:rgb(248,248,248);font-family:Consolas,'Liberation
Mono',Courier,monospace;font-size:12px;white-space:nowrap">Pipe.getPageContext(List
result):</span></div>
Same remarks as Matthias, not sure to understand how it works
(even if I think I've understand ...)</div>
<div dir="ltr"><br>
</div>
</blockquote>
In Java it's back by a WeakHashMap. Lots of Java specific
implementation magic makes it very easy. As I've said otherwise I'm
not in love with the method but it scratches an itch.<br>
<blockquote
cite="mid:CAD_dpu3Pg8EH2PnjmNwU3w1FNoYQM17C=PiUCp-5OMXg4yFZvw@mail.gmail.com"
type="cite">
<div dir="ltr"><br>
</div>
<div dir="ltr">General remarks about Paging:</div>
<div dir="ltr">
Even if it's not in scope for 1.0, I think we have to keep in
mind that we will also be able to do pagination "offline" /
"only client side" (maybe we would do some paging on a Store
element ?) , so we have to consider this when designing our
API.</div>
<div dir="ltr"><br>
</div>
</blockquote>
I've been thinking about that as well. The Stores we have right now
don't lend themselves to paging very well because they don't have a
concept of order built in.<br>
<blockquote
cite="mid:CAD_dpu3Pg8EH2PnjmNwU3w1FNoYQM17C=PiUCp-5OMXg4yFZvw@mail.gmail.com"
type="cite">
<div dir="ltr">Seb</div>
<div dir="ltr"><br>
<div class="gmail_extra"><br>
</div>
<div class="gmail_extra"><font color="#000000" face="Helvetica,
arial, freesans, clean, sans-serif"><span
style="font-size:14px;line-height:22px"><br>
</span></font></div>
<div class="gmail_extra"><font color="#000000" face="Helvetica,
arial, freesans, clean, sans-serif"><span
style="font-size:14px;line-height:22px"><br>
</span></font></div>
<div class="gmail_extra"><br>
</div>
<div class="gmail_extra"><font color="#000000" face="Helvetica,
arial, freesans, clean, sans-serif"><span
style="font-size:14px;line-height:22px"><br>
</span></font>
<div class="gmail_quote">
On Tue, Jan 15, 2013 at 11:37 AM, Matthias Wessendorf <span
dir="ltr"><<a moz-do-not-send="true"
href="mailto:matzew@apache.org" target="_blank">matzew@apache.org</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px
0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><a
moz-do-not-send="true"
href="https://gist.github.com/4532661" target="_blank">https://gist.github.com/4532661</a></blockquote>
</div>
<br>
<br>
</div>
</div>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
aerogear-dev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:aerogear-dev@lists.jboss.org">aerogear-dev@lists.jboss.org</a>
<a class="moz-txt-link-freetext" href="https://lists.jboss.org/mailman/listinfo/aerogear-dev">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a>
</pre>
</blockquote>
<br>
</body>
</html>