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

David Smiley commented on LUCENE-6709:
--------------------------------------

Out of curiosity I looked closer here.  This issue ought to be re-titled 
because, for one thing, RandomAccessOrds doesn't exist anymore as of 
LUCENE-7407 (Lucene 7.0).  SortedSetSelector _used to_ demand that the input 
sortedSet implement RandomAccessOrds but the current code has no equivalent 
constraint.  This suggests maybe things will "just work" if perhaps the the 
check in TrieField that Hoss pointed out were removed.  Indeed, I did that and 
the test Hoss said which used to fail now passes.

> Add RandomAccessOrds support to UninvertedReader's SortedSetDocValues impl 
> (DocTermOrds.Iterator)
> -------------------------------------------------------------------------------------------------
>
>                 Key: LUCENE-6709
>                 URL: https://issues.apache.org/jira/browse/LUCENE-6709
>             Project: Lucene - Core
>          Issue Type: Improvement
>            Reporter: Hoss Man
>            Priority: Major
>
> UninvertedReader's SortedSetDocValues impl doesn't implement the 
> RandomAccessOrds API, so it can't be used with SortedSetSelector.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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

Reply via email to