[ https://issues.apache.org/jira/browse/LUCENE-9902?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17314685#comment-17314685 ]
Gautam Worah commented on LUCENE-9902: -------------------------------------- We could make it to Lucene 8.9 as well. I have submitted a PR for the currentĀ {{lucene}} Github repo. I wonder if we need a specific 8.9 CHANGES.txt entry for this change so that it gets picked up in the 8.9 release? > 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: 10m > 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