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

Jonathan Ellis commented on CASSANDRA-2921:
-------------------------------------------

bq. I think we should stay with Reader/Writer introduced by v2 here to support 
full (expected) BRAF functionality

I don't understand.  Your second patch changes Writer to a Stream which is the 
right direction to take this IMO.  If we're going to stick with Writer and 
Reader both exposing the same RAF interface I don't think there's much point in 
splitting them up.

> Split BufferedRandomAccessFile (BRAF) into Input and Output classes 
> --------------------------------------------------------------------
>
>                 Key: CASSANDRA-2921
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2921
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Pavel Yaskevich
>            Assignee: Pavel Yaskevich
>             Fix For: 1.0
>
>         Attachments: CASSANDRA-2921-make-Writer-a-stream.patch, 
> CASSANDRA-2921-v2.patch, CASSANDRA-2921.patch
>
>
> Split BRAF into Input and Output classes to void complexity related to random 
> I/O in "write" mode that we don't need any more, see CASSANDRA-2879. And make 
> implementation more clean and reusable.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to