[jira] [Commented] (OAK-2722) IndexCopier fails to delete older index directory upon reindex

["Chetan Mehrotra (JIRA)" <ji...@apache.org>]

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

Chetan Mehrotra commented on OAK-2722:
--------------------------------------

For now disabled the test with http://svn.apache.org/r1671795

Would fix this post 1.2 as worst case that directory does not get removed and would need to be removed manually. And then reindex is not done very frequently in prod systems.

> IndexCopier fails to delete older index directory upon reindex
> --------------------------------------------------------------
>
>                 Key: OAK-2722
>                 URL: https://issues.apache.org/jira/browse/OAK-2722
>             Project: Jackrabbit Oak
>          Issue Type: Bug
>          Components: oak-lucene
>            Reporter: Chetan Mehrotra
>            Assignee: Chetan Mehrotra
>            Priority: Minor
>             Fix For: 1.3.0
>
>
> {{IndexCopier}} tries to remove the older index directory incase of reindex. This might fails on platform like Windows if the files are still memory mapped or are locked.
> For deleting directories we would need to take similar approach like being done with deleting old index files i.e. do retries later.
> Due to this following test fails on Windows (Per [~julian.reschke@gmx.de] )
> {noformat}
> Tests run: 9, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.07 sec <<< FAILURE!
> deleteOldPostReindex(org.apache.jackrabbit.oak.plugins.index.lucene.IndexCopierTest)  Time elapsed: 0.02 sec  <<< FAILURE!
> java.lang.AssertionError: Old index directory should have been removed
>         at org.junit.Assert.fail(Assert.java:93)
>         at org.junit.Assert.assertTrue(Assert.java:43)
>         at org.junit.Assert.assertFalse(Assert.java:68)
>         at org.apache.jackrabbit.oak.plugins.index.lucene.IndexCopierTest.deleteOldPostReindex(IndexCopierTest.java:160)
>         at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
>         at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)