[ https://issues.apache.org/jira/browse/LUCENE-1295?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12601340#action_12601340 ]
Otis Gospodnetic commented on LUCENE-1295: ------------------------------------------ I think cosmetic changes are OK if: * they are not mixed with functional changes * there are no patches for the cleaned-up class(es) in JIRA In this case I see only a couple of MLT issues, all of which look like we can take care of them quickly, and then somebody can clean up a little if we feel like it. Anyhow... > Make retrieveTerms(int docNum) public in MoreLikeThis > ----------------------------------------------------- > > Key: LUCENE-1295 > URL: https://issues.apache.org/jira/browse/LUCENE-1295 > Project: Lucene - Java > Issue Type: Improvement > Components: contrib/* > Reporter: Grant Ingersoll > Assignee: Grant Ingersoll > Priority: Trivial > Attachments: LUCENE-1295.patch > > > It would be useful if > {code} > private PriorityQueue retrieveTerms(int docNum) throws IOException { > {code} > were public, since it is similar in use to > {code} > public PriorityQueue retrieveTerms(Reader r) throws IOException { > {code} > It also seems useful to add > {code} > public String [] retrieveInterestingTerms(int docNum) throws IOException{ > {code} > to mirror the one that works on Reader. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]