[ 
https://issues.apache.org/jira/browse/LUCENE-8326?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16503137#comment-16503137
 ] 

Robert Muir commented on LUCENE-8326:
-------------------------------------

I feel the same way as before: we shouldn't split up a class in a 
user-impacting way just because its 1000 lines of code.  To the user it does 
not matter, they just see one class with getters and setters and its easy.

If it really needs to be split up, can we try to do it in a way that doesn't 
impact users, e.g. move some code into package-private implementation classes?

> More Like This Params Refactor
> ------------------------------
>
>                 Key: LUCENE-8326
>                 URL: https://issues.apache.org/jira/browse/LUCENE-8326
>             Project: Lucene - Core
>          Issue Type: Improvement
>          Components: core/query/scoring
>            Reporter: Alessandro Benedetti
>            Priority: Major
>         Attachments: LUCENE-8326.patch, LUCENE-8326.patch, LUCENE-8326.patch
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> More Like This ca be refactored to improve the code readability, test 
> coverage and maintenance.
> Scope of this Jira issue is to start the More Like This refactor from the 
> More Like This Params.
> This Jira will not improve the current More Like This but just keep the same 
> functionality with a refactored code.
> Other Jira issues will follow improving the overall code readability, test 
> coverage and maintenance.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to