Re: svn commit: r1407313 - /lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/store/MockDirectoryWrapper.java

[Robert Muir <rc...@gmail.com>]

my bad, this was always an evil hack.
for now, slow io systems will be unhappy... sorry Uwe.

On Thu, Nov 8, 2012 at 6:34 PM,  <mi...@apache.org> wrote:
> Author: mikemccand
> Date: Thu Nov  8 23:34:59 2012
> New Revision: 1407313
>
> URL: http://svn.apache.org/viewvc?rev=1407313&view=rev
> Log:
> always fsync for now (it's causing test failures otherwise)
>
> Modified:
>     lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/store/MockDirectoryWrapper.java
>
> Modified: lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/store/MockDirectoryWrapper.java
> URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/store/MockDirectoryWrapper.java?rev=1407313&r1=1407312&r2=1407313&view=diff
> ==============================================================================
> --- lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/store/MockDirectoryWrapper.java (original)
> +++ lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/store/MockDirectoryWrapper.java Thu Nov  8 23:34:59 2012
> @@ -162,7 +162,9 @@ public class MockDirectoryWrapper extend
>        throw new IOException("cannot sync after crash");
>      }
>      unSyncedFiles.removeAll(names);
> -    if (LuceneTestCase.rarely(randomState) || delegate instanceof NRTCachingDirectory) {
> +    // TODO: need to improve hack to be OK w/
> +    // RateLimitingDirWrapper in between...
> +    if (true || LuceneTestCase.rarely(randomState) || delegate instanceof NRTCachingDirectory) {
>        // don't wear out our hardware so much in tests.
>        delegate.sync(names);
>      }
>
>

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