[
https://issues.apache.org/jira/browse/LUCENE-10467?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17517557#comment-17517557
]
ASF subversion and git services commented on LUCENE-10467:
----------------------------------------------------------
Commit d1bd59f6bc6effe04f802e0d9a8f88fc7920f2cd in lucene's branch
refs/heads/branch_9x from Yuting Gan
[ https://gitbox.apache.org/repos/asf?p=lucene.git;h=d1bd59f6bc6 ]
LUCENE-10467: Throws IllegalArgumentException for getAllDims and getTopChildren
if topN <= 0
> Throws IllegalArgumentException for getAllDims and getTopChildren if topN <= 0
> ------------------------------------------------------------------------------
>
> Key: LUCENE-10467
> URL: https://issues.apache.org/jira/browse/LUCENE-10467
> Project: Lucene - Core
> Issue Type: Improvement
> Components: modules/facet
> Reporter: Yuting Gan
> Priority: Minor
> Time Spent: 1.5h
> Remaining Estimate: 0h
>
> Currently, there are different behaviors from subclass that implements and
> overrides getAllDims and getTopChildren when passing in an invalid TopN
> parameter (topN <= 0). Some overridden implementations throw a
> NullPointerException, some throw an IllegalArgumentException, and others
> throw no exception.
> It would provide a better user experience by consistently throwing an
> IllegalArgumentException when requesting topN <= 0 for these two
> functionalities across all implementations.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org
