[
https://issues.apache.org/jira/browse/CASSANDRA-3157?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13100382#comment-13100382
]
Hudson commented on CASSANDRA-3157:
-----------------------------------
Integrated in Cassandra #1089 (See
[https://builds.apache.org/job/Cassandra/1089/])
After a "short read", the wrong read command may be used
patch by slebresne; reviewed by jbellis for CASSANDRA-3157
slebresne :
http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1166716
Files :
* /cassandra/trunk/CHANGES.txt
* /cassandra/trunk/src/java/org/apache/cassandra/service/StorageProxy.java
> After a "short read", the wrong read command may be used
> --------------------------------------------------------
>
> Key: CASSANDRA-3157
> URL: https://issues.apache.org/jira/browse/CASSANDRA-3157
> Project: Cassandra
> Issue Type: Bug
> Components: Core
> Affects Versions: 1.0
> Reporter: Sylvain Lebresne
> Assignee: Sylvain Lebresne
> Priority: Minor
> Fix For: 1.0
>
> Attachments: 3157.patch
>
>
> In fetchRows, there is this code:
> {noformat}
> for (int i = 0; i < commandsToSend.size(); i++)
> {
> ReadCallback<Row> handler = readCallbacks.get(i);
> ReadCommand command = commands.get(i);
> {noformat}
> On the first iteration of fetchRows, commands == commandsToSend so this is
> ok, but on a short read, commandsToSend will only contain the command to
> retry so we'll pick up the wrong command on the last line.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
