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

Dawid Weiss commented on LUCENE-6239:
-------------------------------------

A similar argument could be made about the absolute need to compile and 
restrict the core to compact1 profile... why would you need that in a search 
library if it adds like a few milliseconds time at startup (once)?

The code isn't useless, but it may be irrelevant. If there are any places in 
the code that measure reference arrays then I am for keeping the estimation 
heuristics in, they provide value. If there are no places doing that (other 
than logging) then I'd remove it.

> Remove RAMUsageEstimator Unsafe calls
> -------------------------------------
>
>                 Key: LUCENE-6239
>                 URL: https://issues.apache.org/jira/browse/LUCENE-6239
>             Project: Lucene - Core
>          Issue Type: Bug
>            Reporter: Robert Muir
>
> This is unnecessary risk. We should remove this stuff, it is not needed here. 
> We are a search engine, not a ram calculator.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to