[ 
https://issues.apache.org/jira/browse/CASSANDRA-950?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Johan Oskarsson updated CASSANDRA-950:
--------------------------------------

    Attachment: CASSANDRA-950.patch

Updated patch that caches the length on init for read only files.

Updated if ( position. My brain doesn't even notice if there is a space between 
if and (. I think I'm used to and prefer not having a space before ( since 
that's the norm after method names.

> Cache BufferedRandomAccessFile.length
> -------------------------------------
>
>                 Key: CASSANDRA-950
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-950
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Johan Oskarsson
>         Attachments: CASSANDRA-950.patch, CASSANDRA-950.patch
>
>
> Profiling shows that BufferedRandomAccessFile.length is eating up an 
> unexpected amount of cpu. It believe it is the call through the "native 
> barrier" that is causing it. We often call the length method via is end of 
> file when reading data. We can add caching of the file length to avoid this 
> call most times.

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

Reply via email to