[
https://issues.apache.org/jira/browse/CASSANDRA-2921?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13069457#comment-13069457
]
Pavel Yaskevich commented on CASSANDRA-2921:
--------------------------------------------
bq. 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.
Second patch used to have random I/O writer but my point is - if we want to
support BRAF lets do both Reader and Writer and do SequentialWriter in the
separate file. But if you really think we should remove write functionality
from BRAF, I'm okay with that, will attach v4 patch with some minor changes to
your v3 soon.
> 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: 2921-v3.txt, 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
