Re: org.apache.cassandra.service.ReadResponseResolver question

2010-12-15 Thread Daniel Doubleday
On Dec 14, 2010, at 9:20 PM, Jonathan Ellis wrote: Correct. https://issues.apache.org/jira/browse/CASSANDRA-1830 is open to fix that. If you'd like to review the patch there, that would be very helpful. :) That patch looks good to me :-) Should have checked jira first ... Speaking of

Re: org.apache.cassandra.service.ReadResponseResolver question

2010-12-15 Thread Jonathan Ellis
That sounds like an interesting patch (as you point out we have had #982 open for a while), but I don't think we want to do something relatively invasive on the 0.6 branch. Let's target 0.7 or trunk. On Wed, Dec 15, 2010 at 11:23 AM, Daniel Doubleday daniel.double...@gmx.net wrote: On Dec

org.apache.cassandra.service.ReadResponseResolver question

2010-12-14 Thread Daniel Doubleday
Hi I'm sorry - don't want to be a pain in the neck with source questions. So please just ignore me if this is stupid: Isn't org.apache.cassandra.service.ReadResponseResolver suposed to throw a DigestMismatchException if it receives a digest wich does not match the digest of a read message?

Re: org.apache.cassandra.service.ReadResponseResolver question

2010-12-14 Thread Jonathan Ellis
Correct. https://issues.apache.org/jira/browse/CASSANDRA-1830 is open to fix that. If you'd like to review the patch there, that would be very helpful. :) On Tue, Dec 14, 2010 at 1:55 PM, Daniel Doubleday daniel.double...@gmx.netwrote: Hi I'm sorry - don't want to be a pain in the neck with