Re: [infinispan-dev] Hot Rod protocol design at CR stage. Feedback welcome

On 25 Mar 2010, at 13:43, Manik Surtani wrote: > > On 25 Mar 2010, at 11:27, Galder Zamarreno wrote: > >> Mircea/Manik, >> >> There's a final thing I'm not too happy about and that's the response >> to >> the stats command. In the current form, the reply to a paramaterless >> stats >> command that returns all stats available looks like this: >> >> [header][name1 length][name1][value1 length][value1] >> [header][name2 length][name2][value2 length][value2] >> ... >> [header] -> This would be the end marker >> >> This looks quite wasteful since we're adding a header per each stat. >> Instead I suggest doing: >> >> [header][number of stats][name1 length][name1][value1 >> length][value1][name2 length][name2][value2 length][value2]... > > +1 +1 > >> >> This is less wasteful and having the number of stats at the beginning >> signals how much to read. >> >> WDYT? >> >> On Mon, 22 Mar 2010 16:48:26 +0100, Galder Zamarreno >> <galder(a)redhat.com&gt; >> wrote: >> >>> Hi all, >>> >>> After having yet another round of discussions, we consider the Hot Rod >>> protocol design at CR stage. Here's the final results: >>> http://community.jboss.org/wiki/HotRodProtocol >>> >>> While me and Mircea are busy coding both the server and client, please >>> take some time to have a read through it and let us know what you >>> think. >>> >>> By the way, remember that we've deferred certain functionality for the >>> moment as indicated in a previous email: >>> [ISPN-375] Enable Hot Rod clients to start transactions [Open, Major, >>> Galder Zamarreno] http://jira.jboss.org/jira/browse/ISPN-375 >>> [ISPN-374] Add event handling to Hot Rod [Open, Major, Galder >>> Zamarreno] >>> http://jira.jboss.org/jira/browse/ISPN-374 >>> >>> Cheers, >> >> >> -- >> Galder Zamarreño >> Sr. Software Engineer >> Infinispan, JBoss Cache >> >> _______________________________________________ >> infinispan-dev mailing list >> infinispan-dev(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/infinispan-dev > > -- > Manik Surtani > manik(a)jboss.org > Lead, Infinispan > Lead, JBoss Cache > http://www.infinispan.org > http://www.jbosscache.org > > > > > > _______________________________________________ > infinispan-dev mailing list > infinispan-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/infinispan-dev _______________________________________________ infinispan-dev mailing list infinispan-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev