[
https://issues.apache.org/jira/browse/LUCENE-9902?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17367247#comment-17367247
]
Michael McCandless commented on LUCENE-9902:
--------------------------------------------
Thanks for catching this [~mayya]! But shouldn't we just backport this change
(to 8.10.0) now? Did it break any APIs? Isn't the switch from taking
{{String[]}} to {{String...}} backwards compatible?
> Update faceting API to use modern Java features
> -----------------------------------------------
>
> Key: LUCENE-9902
> URL: https://issues.apache.org/jira/browse/LUCENE-9902
> Project: Lucene - Core
> Issue Type: Improvement
> Components: modules/facet
> Reporter: Gautam Worah
> Priority: Minor
> Time Spent: 1h 40m
> Remaining Estimate: 0h
>
> I was using the {{public int getOrdinal(String dim, String[] path)}} API for
> a single {{path}} String and found myself creating an array with a single
> element. We can start using variable length args for this method.
> I also propose this change:
> I wanted to know the specific count of an ordinal using using the
> {{getValue}} API from {{IntTaxonomyFacets}} but the method is private. It
> would be good if we could change it to {{protected}} so that users can know
> the value of an ordinal without looking up the {{FacetLabel}} and then
> checking its value.
--
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
