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

Robert Muir commented on LUCENE-5373:
-------------------------------------

can we please use AtomicLong rather than volatile?

> Lucene42DocValuesProducer.ramBytesUsed is over-estimated
> --------------------------------------------------------
>
>                 Key: LUCENE-5373
>                 URL: https://issues.apache.org/jira/browse/LUCENE-5373
>             Project: Lucene - Core
>          Issue Type: Bug
>            Reporter: Adrien Grand
>            Assignee: Adrien Grand
>            Priority: Minor
>         Attachments: LUCENE-5373.patch, LUCENE-5373.patch
>
>
> Lucene42DocValuesProducer.ramBytesUsed uses 
> {{RamUsageEstimator.sizeOf(this)}} to return an estimation of the memory 
> usage. One of the issues (there might be other ones) is that this class has a 
> reference to an IndexInput that might link to other data-structures that we 
> wouldn't want to take into account. For example, index inputs of a 
> {{RAMDirectory}} all point to the directory itself, so 
> {{Lucene42DocValuesProducer.ramBytesUsed}} would return the amount of memory 
> used by the whole directory.



--
This message was sent by Atlassian JIRA
(v6.1.4#6159)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to