[ 
https://issues.apache.org/jira/browse/LUCENE-10467?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yuting Gan updated LUCENE-10467:
--------------------------------
    Description: 
Currently, there are different behaviors from subclass that implements 
`getAllDims` and `getTopChildren` when passing in an invalid TopN parameter 
(`topN <= 0`). Some overridden implementations throw a `NullPointerException`, 
some throw an `IllegalArgumentException`, and others do not throw any exception.

It would provide a better user experience by consistently throwing an 
IllegalArgumentException when requesting topN <= 0 for these two 
functionalities across all implementations.

  was:
Currently, there are different behaviors from subclass that implements 
`getAllDims` and `getTopChildren` when passing in an invalid TopN parameter 
(`topN <= 0`). Some overridden implementations throw a `NullPointerException`, 
some throw an `IllegalArgumentException`, and others do not throw any exception.

It would provide better user experience by consistently throwing an 
IllegalArgumentException when requesting topN <= 0 for these two 
functionalities across all implementations.


> 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: 10m
>  Remaining Estimate: 0h
>
> Currently, there are different behaviors from subclass that implements 
> `getAllDims` and `getTopChildren` when passing in an invalid TopN parameter 
> (`topN <= 0`). Some overridden implementations throw a 
> `NullPointerException`, some throw an `IllegalArgumentException`, and others 
> do not throw any 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

Reply via email to