[jira] [Commented] (LUCENE-9476) Add a bulk ordinal->FacetLabel API

2021-09-02 Thread ASF subversion and git services (Jira)


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

ASF subversion and git services commented on LUCENE-9476:
-

Commit 1036c708db743342abd17e5f568b4f78587a80fa in lucene's branch 
refs/heads/main from Gautam Worah
[ https://gitbox.apache.org/repos/asf?p=lucene.git;h=1036c70 ]

LUCENE-9476: Add getBulkPath API to DirectoryTaxonomyReader for faster ordinal 
-> FacetLabel lookup (#179)

Co-authored-by: Gautam Worah 

> Add a bulk ordinal->FacetLabel API
> --
>
> Key: LUCENE-9476
> URL: https://issues.apache.org/jira/browse/LUCENE-9476
> Project: Lucene - Core
>  Issue Type: Improvement
>  Components: modules/facet
>Affects Versions: 8.6.1
>Reporter: Gautam Worah
>Priority: Minor
>  Labels: performance
>  Time Spent: 12h
>  Remaining Estimate: 0h
>
> This issue is a spillover from the 
> [PR|https://github.com/apache/lucene-solr/pull/1733/files] for LUCENE 9450
> The idea here is to share a single {{BinaryDocValues}} instance per leaf per 
> query instead of creating a new one each time in the 
> {{DirectoryTaxonomyReader}}.
> Suggested by [~mikemccand]
>  
>  
>  



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



[jira] [Commented] (LUCENE-9476) Add a bulk ordinal->FacetLabel API

2021-01-25 Thread Gautam Worah (Jira)


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

Gautam Worah commented on LUCENE-9476:
--

Submitted a WIP PR [here|https://github.com/apache/lucene-solr/pull/2246]

> Add a bulk ordinal->FacetLabel API
> --
>
> Key: LUCENE-9476
> URL: https://issues.apache.org/jira/browse/LUCENE-9476
> Project: Lucene - Core
>  Issue Type: Improvement
>  Components: modules/facet
>Affects Versions: 8.6.1
>Reporter: Gautam Worah
>Priority: Minor
>  Labels: performance
>
> This issue is a spillover from the 
> [PR|https://github.com/apache/lucene-solr/pull/1733/files] for LUCENE 9450
> The idea here is to share a single {{BinaryDocValues}} instance per leaf per 
> query instead of creating a new one each time in the 
> {{DirectoryTaxonomyReader}}.
> Suggested by [~mikemccand]
>  
>  
>  



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



[jira] [Commented] (LUCENE-9476) Add a bulk ordinal->FacetLabel API

2020-08-24 Thread Michael McCandless (Jira)


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

Michael McCandless commented on LUCENE-9476:


When a query computes facets, in the end it gets a set of `integer` ordinals 
and counts, and must resolve those ordinals to human friendly `FacetLabels`.

There might be 10s of such lookups.

With the change in LUCENE-9450, we are creating a new `BinaryDocValues` for 
every `ord to FacetLabel` lookup, but for this issue we should add a new API 
that takes `int[] ordinals` or so, and then returns `FacetLabel[]`, by first 
collating all ordinals into the segments (in the taxonomy index) and then 
creating one `BinaryDocValues` per segment and looking up the ordinals 
sequentially.  This should be a sizable speedup over what we are doing in 
LUCENE-9450, which is already also a bit faster than what we do in mainline now 
(use stored fields).

> Add a bulk ordinal->FacetLabel API
> --
>
> Key: LUCENE-9476
> URL: https://issues.apache.org/jira/browse/LUCENE-9476
> Project: Lucene - Core
>  Issue Type: Improvement
>  Components: modules/facet
>Affects Versions: 8.6.1
>Reporter: Gautam Worah
>Priority: Minor
>  Labels: performance
>
> This issue is a spillover from the 
> [PR|https://github.com/apache/lucene-solr/pull/1733/files] for LUCENE 9450
> The idea here is to share a single {{BinaryDocValues}} instance per leaf per 
> query instead of creating a new one each time in the 
> {{DirectoryTaxonomyReader}}.
> Suggested by [~mikemccand]
>  
>  
>  



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