[
https://issues.apache.org/jira/browse/LUCENE-5695?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14006096#comment-14006096
]
Robert Muir commented on LUCENE-5695:
-------------------------------------
I think if we want to add a default impl, it should return -1 or throw UOE
Otherwise there is no differentiation between "not implemented" and "doesnt use
space"
> Add DocIdSet.ramBytesUsed
> -------------------------
>
> Key: LUCENE-5695
> URL: https://issues.apache.org/jira/browse/LUCENE-5695
> Project: Lucene - Core
> Issue Type: New Feature
> Affects Versions: 4.9, 5.0
> Reporter: Adrien Grand
> Assignee: Adrien Grand
> Priority: Minor
> Attachments: LUCENE-5695.patch
>
>
> LUCENE-5463 tried to remove calls to {{RamUsageEstimator.sizeOf(Object)}} yet
> it was not always possible to remove the call when there was no other API to
> compute the memory usage of a particular class. In particular, this is the
> case for {{CachingWrapperFilter.sizeInBytes()}} that needs to be able to get
> the memory usage of any cacheable {{DocIdSet}} instance.
> We could add {{DocIdSet.ramBytesUsed}} in order to remove the need for
> {{RamUsageEstimator}}. This will also help have bounded filter caches and
> take the size of the cached doc id sets into account when doing evictions.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
