[infinispan-dev] Hot Rod - pt2

[Manik Surtani]

A few comments:

- Why do you have OpCode in your response header?  Surely this is redundant?  If the client is sync, it knows what it sent.  If it is async, it has a message ID.
- 'Not So Dumb' and 'Clever' response headers should be optional?  Surely this stuff is only sent when there is a topology change?  We also may need some extra info here - how does the back-end know to send this info?  If a client hits different back-end nodes, and there is a topology change, how does Node A decide that it should not bother with topology info since the client already hit Node B after the topo change and has the new topology map?  Perhaps a TopologyVersion (== JGroups View ID) should be sent back with any topo map, and the client would send it's current TopologyVersion with every request (non-dumb clients only)?  Could be a vlong... 

Cheers
Manik


On 14 Dec 2009, at 20:08, Galder Zamarreno wrote:

> Hi all,
> 
> Re: http://community.jboss.org/wiki/HotRodProtocol
> 
> I've updated the wiki with the following stuff:
> - Renamed replaceIfEquals to replaceIfUnmodified
> - Added remove and removeIfUnmodified.
> - Added containsKey command.
> - Added getWithCas command so that cas value can be returned. I decided 
> for a separate command rather than adding cas to get return because you 
> don't always want cas to be returned. Having a separate command makes 
> better use of network bandwith.
> - Added stats command. JMX attributes are basically accessible through 
> this, including cache size.
> - Added error handling section and updated status codes.
> 
> Note that Mircea added some interesting comments and I replied to them 
> directly in the wiki.
> 
> Still remaining to add:
> - Commands: putForExternalRead evict, clear, version, name and quite 
> commands.
> - Passing flags.
> 
> Regards,
> 
> p.s. Updating this has been quite a struggle due to F12 + FF 3.5.5 
> crashing at least 5 times, plus parts of the wiki dissapearing after 
> publishing them!
> -- 
> 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