[geode] branch develop updated: GEODE-3926: handle new lucene indexing exception

[nnag]

This is an automated email from the ASF dual-hosted git repository.

nnag pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/develop by this push:
     new b71b79f  GEODE-3926: handle new lucene indexing exception
b71b79f is described below

commit b71b79fe813b408fc50a172644007e2d5f54a1e2
Author: nabarun <n...@cs.wisc.edu>
AuthorDate: Thu Apr 12 13:07:15 2018 -0700

    GEODE-3926: handle new lucene indexing exception
    
        * Ignored test till LUCENE_REINDEX flag is removed
        * Indexing in progress exception will not be thrown if LUCENE_REINDEX 
flag is not enabled
---
 .../geode/cache/lucene/internal/LuceneIndexForPartitionedRegion.java   | 3 ++-
 .../cache/lucene/internal/LuceneIndexForPartitionedRegionTest.java     | 2 ++
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git 
a/geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/LuceneIndexForPartitionedRegion.java
 
b/geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/LuceneIndexForPartitionedRegion.java
index 247e1b4..99b9a89 100644
--- 
a/geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/LuceneIndexForPartitionedRegion.java
+++ 
b/geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/LuceneIndexForPartitionedRegion.java
@@ -209,7 +209,8 @@ public class LuceneIndexForPartitionedRegion extends 
LuceneIndexImpl {
   @Override
   public boolean isIndexAvailable(int id) {
     PartitionedRegion fileAndChunkRegion = getFileAndChunkRegion();
-    return 
fileAndChunkRegion.get(IndexRepositoryFactory.APACHE_GEODE_INDEX_COMPLETE, id) 
!= null;
+    return 
(fileAndChunkRegion.get(IndexRepositoryFactory.APACHE_GEODE_INDEX_COMPLETE, id) 
!= null
+        || !LuceneServiceImpl.LUCENE_REINDEX);
   }
 
   private void destroyOnRemoteMembers() {
diff --git 
a/geode-lucene/src/test/java/org/apache/geode/cache/lucene/internal/LuceneIndexForPartitionedRegionTest.java
 
b/geode-lucene/src/test/java/org/apache/geode/cache/lucene/internal/LuceneIndexForPartitionedRegionTest.java
index 3a36a94..05d960d 100644
--- 
a/geode-lucene/src/test/java/org/apache/geode/cache/lucene/internal/LuceneIndexForPartitionedRegionTest.java
+++ 
b/geode-lucene/src/test/java/org/apache/geode/cache/lucene/internal/LuceneIndexForPartitionedRegionTest.java
@@ -19,6 +19,7 @@ import static org.mockito.Mockito.*;
 
 import java.util.Properties;
 
+import org.junit.Ignore;
 import org.junit.Rule;
 import org.junit.Test;
 import org.junit.experimental.categories.Category;
@@ -90,6 +91,7 @@ public class LuceneIndexForPartitionedRegionTest {
     assertTrue(index.fileRegionExists(fileRegionName));
   }
 
+  @Ignore // Enable the test when LuceneServiceImpl.LUCENE_REINDEX feature 
flag is removed.
   @Test
   public void indexIsAvailableReturnsFalseIfCompleteFileIsNotPresent() {
     String name = "indexName";

-- 
To stop receiving notification emails like this one, please contact
n...@apache.org.