[ https://issues.apache.org/jira/browse/LUCENE-1402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12633821#action_12633821 ]
Uwe Schindler commented on LUCENE-1402: --------------------------------------- And I found a bug in CheckIndex.checkIndex(): the boolean parameter "fix" is not used, you must always manually call fix() after checking, so this parameter is useless! > CheckIndex API changed without backwards compaitibility > ------------------------------------------------------- > > Key: LUCENE-1402 > URL: https://issues.apache.org/jira/browse/LUCENE-1402 > Project: Lucene - Java > Issue Type: Bug > Components: Index > Affects Versions: 2.4 > Reporter: Uwe Schindler > Assignee: Michael McCandless > Priority: Minor > Attachments: LUCENE-1402.patch > > > The API of CheckIndex changed. The Check function returns a CheckIndexStatus > and not boolean. And JavaDocs notes the boolean return value. > I am not sure if it works, but it would be good to have the check method that > returns boolean available @Deprecated, i.e. > @Deprecated public static CheckIndexStatus check(Directory dir, boolean > doFix) throws IOException { > final CheckIndexStatus stat=this.check(dir,doFix); > return stat.clean; > } > I am not sure, if it can be done with the same method name, but it prevents > drop-in-replacements of Lucene to work. -- 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]