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

[Mircea Markus]

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 at redhat.com>  
>> 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 at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/infinispan-dev
> 
> --
> Manik Surtani
> manik at jboss.org
> Lead, Infinispan
> Lead, JBoss Cache
> http://www.infinispan.org
> http://www.jbosscache.org
> 
> 
> 
> 
> 
> _______________________________________________
> infinispan-dev mailing list
> infinispan-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/infinispan-dev