lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Uwe Schindler (JIRA)" <>
Subject [jira] Updated: (LUCENE-1402) CheckIndex API changed without backwards compaitibility
Date Tue, 23 Sep 2008 19:17:44 GMT


Uwe Schindler updated LUCENE-1402:

    Attachment: LUCENE-1402-uwe.patch

I hope you did not start to do a patch. I created one, that contains all the comments and
fixes also some bugs:
- the parameter fix in static method remove
- if infoStream/out==null NPE's at different positions

It is not completely tested, but it compiles correct and works for me (using the API, not
So it needs intensive testing, but should be OK.

Maybe rename method fix() to fixIndex() in comparison to checkIndex().

> CheckIndex API changed without backwards compaitibility
> -------------------------------------------------------
>                 Key: LUCENE-1402
>                 URL:
>             Project: Lucene - Java
>          Issue Type: Bug
>          Components: Index
>    Affects Versions: 2.4
>            Reporter: Uwe Schindler
>            Assignee: Michael McCandless
>            Priority: Minor
>         Attachments: LUCENE-1402-uwe.patch, 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:
For additional commands, e-mail:

View raw message