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

Ankur edited comment on LUCENE-9444 at 9/29/20, 12:05 AM:
----------------------------------------------------------

Thanks [~mikemccand] for merging the 
[PR-1893.|https://github.com/apache/lucene-solr/pull/1893/files]

I just realized that the changes in {{TestTaxonomyFacetCounts.testRandom()}}  
did not exercise the API to get facet labels for specific dimension -  
{{TaxonomyFacetLabels.nextFacetLabel(docId, facetDimension)}} so I added the 
required changes in 
[PR-1928.|https://github.com/apache/lucene-solr/pull/1928/files]

 

Re-opening the issue so that you can take a look.


was (Author: goankur):
Thanks [~mikemccand] for merging the 
[PR-1893.|https://github.com/apache/lucene-solr/pull/1893/files]

I just realized that the changes in {{TestTaxonomyFacetCounts.testRandom()}}  
did not exercise the API to get facet labels for specific dimension -  
{{TaxonomyFacetLabels.nextFacetLabel(docId, facetDimension)}} so I made changes 
in [PR-1928.|https://github.com/apache/lucene-solr/pull/1928/files] Can you 
please take a look ?

> Need an API to easily fetch facet labels for a field in a document
> ------------------------------------------------------------------
>
>                 Key: LUCENE-9444
>                 URL: https://issues.apache.org/jira/browse/LUCENE-9444
>             Project: Lucene - Core
>          Issue Type: Improvement
>          Components: modules/facet
>    Affects Versions: 8.6
>            Reporter: Ankur
>            Priority: Major
>              Labels: facet
>             Fix For: master (9.0), 8.7
>
>         Attachments: LUCENE-9444.patch, LUCENE-9444.patch, 
> LUCENE-9444.v2.patch
>
>          Time Spent: 4.5h
>  Remaining Estimate: 0h
>
> A facet field may be included in the list of fields whose values are to be 
> returned for each hit.
> In order to get the facet labels for each hit we need to
>  # Create an instance of _DocValuesOrdinalsReader_ and invoke 
> _getReader(LeafReaderContext context)_ method to obtain an instance of 
> _OrdinalsSegmentReader()_
>  # _OrdinalsSegmentReader.get(int docID, IntsRef ordinals)_ method is then 
> used to fetch and decode the binary payload in the document's BinaryDocValues 
> field. This provides the ordinals that refer to facet labels in the 
> taxonomy.**
>  # Lastly TaxonomyReader.getPath(ord) is used to fetch the labels to be 
> returned.
>  
> Ideally there should be a simple API - *String[] getLabels(docId)* that hides 
> all the above details and gives us the string labels. This can be part of 
> *TaxonomyFacets* but that's just one idea.
> I am opening this issue to get community feedback and suggestions.
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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

Reply via email to