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

Robert Muir commented on LUCENE-6308:
-------------------------------------

Alan is correct. Mike, the issue is you don't need 2.1B tokens to hit this. And 
I don't know where "its a reserved value" comes from. At least nothing is 
reserving it, in indexwriter or elsewhere. Instead we just check for 
negatives/overflow.

I've seen people use large posInc gaps between fields. This can make huge 
position numbers. Also if someone forgets clearAttributes the positions grow 
exponentially. Sure its bad, but for small docs i bet plenty of people have 
HUGE positions and don't realize it.

Can we just change the constant to -2? Then the problem is solved.

> SpansEnum, deprecate Spans
> --------------------------
>
>                 Key: LUCENE-6308
>                 URL: https://issues.apache.org/jira/browse/LUCENE-6308
>             Project: Lucene - Core
>          Issue Type: Bug
>          Components: core/search
>    Affects Versions: Trunk
>            Reporter: Paul Elschot
>            Priority: Minor
>         Attachments: LUCENE-6308-changeapi.patch, 
> LUCENE-6308-changeapi.patch, LUCENE-6308-changeapi.patch, 
> LUCENE-6308-changeapi.patch, LUCENE-6308-changeapi.patch, LUCENE-6308.patch, 
> LUCENE-6308.patch, LUCENE-6308.patch, LUCENE-6308.patch, LUCENE-6308.patch, 
> LUCENE-6308.patch
>
>
> An alternative for Spans that looks more like PositionsEnum and adds two 
> phase doc id iteration



--
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