diff --git a/querydsl-lucene/src/main/java/com/mysema/query/lucene/session/impl/FileLockingWriter.java b/querydsl-lucene/src/main/java/com/mysema/query/lucene/session/impl/FileLockingWriter.java index b4283e3b3..8db836132 100644 --- a/querydsl-lucene/src/main/java/com/mysema/query/lucene/session/impl/FileLockingWriter.java +++ b/querydsl-lucene/src/main/java/com/mysema/query/lucene/session/impl/FileLockingWriter.java @@ -34,8 +34,9 @@ public class FileLockingWriter implements LuceneWriter { public FileLockingWriter(Directory directory, boolean createNew, long defaultLockTimeout, ReleaseListener releaseListener) { IndexWriter.setDefaultWriteLockTimeout(defaultLockTimeout); + boolean create = createNew; try { - if (createNew == false) { + if (!create) { try { writer = new IndexWriter(directory, new StandardAnalyzer(Version.LUCENE_30), false, @@ -44,17 +45,17 @@ public class FileLockingWriter implements LuceneWriter { } catch (FileNotFoundException e) { // Convience to create a new index if it's not already // existing - createNew = true; + create = true; } } - if (createNew == true) { + if (create) { writer = new IndexWriter(directory, new StandardAnalyzer(Version.LUCENE_30), true, MaxFieldLength.LIMITED); } } catch (LockObtainFailedException e) { - logger.error("Got timeout " + System.currentTimeMillis()); + logger.error("Got lock timeout "); throw new WriteLockObtainFailedException(e); } catch (IOException e) { throw new QueryException(e); diff --git a/querydsl-lucene/src/main/java/com/mysema/query/lucene/session/impl/LuceneSessionHolder.java b/querydsl-lucene/src/main/java/com/mysema/query/lucene/session/impl/LuceneSessionHolder.java index 3c21b12aa..505506ae4 100644 --- a/querydsl-lucene/src/main/java/com/mysema/query/lucene/session/impl/LuceneSessionHolder.java +++ b/querydsl-lucene/src/main/java/com/mysema/query/lucene/session/impl/LuceneSessionHolder.java @@ -32,7 +32,7 @@ public final class LuceneSessionHolder { private final boolean readOnly; - public TransactionalScope(boolean readOnly) { + private TransactionalScope(boolean readOnly) { this.readOnly = readOnly; } }