[
https://issues.apache.org/jira/browse/LUCENE-9902?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17314176#comment-17314176
]
Greg Miller commented on LUCENE-9902:
-------------------------------------
+1. It looks like the increment methods in IntTaxonomyFacets are already
protected, so making getValue protected as well seems pretty reasonable (and
more consistent).
> 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
>
> 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
