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

Sylvain Lebresne commented on CASSANDRA-9499:
---------------------------------------------

For the records, the initial intent was to actually use the existing 
{{EncodedDataInputStream/EncodedDataOutputStream}} (but I haven't had time to 
wire it in 8099 yet), and that's an alternative option.

That said, I do prefer the idea of adding explicit new methods for integer 
encoding (if only because we have better control over which integer we actually 
encode). But I believe this mean we'll also need some {{DataInputPlus}} 
interface with a {{readVInt}}. And if we do that, it'd be nice to replace the 
existing use of {{EncodedData*}} for the sake of not having multiple ways to do 
the same thing.


> Introduce writeVInt method to DataOutputStreamPlus
> --------------------------------------------------
>
>                 Key: CASSANDRA-9499
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-9499
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Benedict
>            Assignee: Ariel Weisberg
>            Priority: Minor
>             Fix For: 3.0 beta 1
>
>
> CASSANDRA-8099 really could do with a writeVInt method, for both fixing 
> CASSANDRA-9498 but also efficiently encoding timestamp/deletion deltas. It 
> should be possible to make an especially efficient implementation against 
> BufferedDataOutputStreamPlus.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to