You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Jonathan Ellis (JIRA)" <ji...@apache.org> on 2010/06/19 16:10:23 UTC

[jira] Commented: (CASSANDRA-1200) additional BRAF tests

    [ https://issues.apache.org/jira/browse/CASSANDRA-1200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12880485#action_12880485 ] 

Jonathan Ellis commented on CASSANDRA-1200:
-------------------------------------------

        if (len > b.length-off)
            throw new IllegalArgumentException("byte[] does not have space for " + len + " bytes (len > byte[].length-offset)");

this is wrong, the read() contract is

"Reads up to len bytes of data from this file into an array of bytes... returns the total number of bytes read into the buffer"

> additional BRAF tests
> ---------------------
>
>                 Key: CASSANDRA-1200
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-1200
>             Project: Cassandra
>          Issue Type: Improvement
>    Affects Versions: 0.7
>            Reporter: Matthew F. Dennis
>            Assignee: Matthew F. Dennis
>            Priority: Minor
>             Fix For: 0.7
>
>         Attachments: trunk-1200-v1.txt
>
>
> In comments/IRC there was some concern about BRAF.readFully() raising EOFException when appropriate

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.