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

Robert Muir commented on SOLR-2928:
-----------------------------------

I tried to enable this again (by disabling this test), but at least one DIH 
test has issues here too.

The thing that stumps me is how these tests manage to bypass 
MockDirectoryWrapper's 'still has open locks' assertion.
Something sneaky is happening wrt to MockLockWrapper and 
CachingDirectoryFactory... this should be failing rather
than finding it later when asserting unref'ed files on close.




                
> TestIndexingPerformance leaves a write.lock after it finishes
> -------------------------------------------------------------
>
>                 Key: SOLR-2928
>                 URL: https://issues.apache.org/jira/browse/SOLR-2928
>             Project: Solr
>          Issue Type: Bug
>            Reporter: Robert Muir
>
> we had to add a boolean to allow solr tests to explicitly avoid additional 
> checks because of this, which is bad.
> To expose the bug, just do:
> {noformat}
> Index: 
> solr/test-framework/src/java/org/apache/solr/core/MockDirectoryFactory.java
> ===================================================================
> --- 
> solr/test-framework/src/java/org/apache/solr/core/MockDirectoryFactory.java   
>     (revision 1207870)
> +++ 
> solr/test-framework/src/java/org/apache/solr/core/MockDirectoryFactory.java   
>     (working copy)
> @@ -31,12 +31,6 @@
>  
>    @Override
>    public Directory create(String path) throws IOException {
> -    MockDirectoryWrapper dir = LuceneTestCase.newFSDirectory(new File(path));
> -    // Somehow removing unref'd files in Solr tests causes
> -    // problems... there's some interaction w/
> -    // CachingDirectoryFactory.  Once we track down where Solr
> -    // isn't closing an IW, we can re-enable this:
> -    dir.setAssertNoUnrefencedFilesOnClose(false);
> -    return dir;
> +    return LuceneTestCase.newFSDirectory(new File(path));
>    }
>  }
> {noformat}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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

Reply via email to