HSEARCH-471 Disable dirty checking optimization when interceptor is used

hibernate · Feb 28, 2012 · c645193 · c645193
1 parent 7f74e28
commit c645193
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/hibernate-search-engine/src/main/java/org/hibernate/search/spi/SearchFactoryBuilder.java b/hibernate-search-engine/src/main/java/org/hibernate/search/spi/SearchFactoryBuilder.java
@@ -407,7 +407,10 @@ private void initDocumentBuilders(SearchConfiguration cfg, BuildContext buildCon
 
 			Class mappedClass = classMappings.get( mappedXClass );
 			MutableEntityIndexBinding mappedEntity = indexesFactory.buildEntityIndexBinding( mappedXClass, mappedClass, cfg, buildContext );
-
+			//interceptor might use non indexed state
+		    if ( mappedEntity.getIndexingActionInterceptor() != null ) {
+				optimizationBlackListedTypes.add( mappedXClass );
+			}
 			// Create all DocumentBuilderIndexedEntity
 			//FIXME DocumentBuilderIndexedEntity needs to be built by a helper method receiving Class<T> to infer T properly
 			//XClass unfortunately is not (yet) genericized: TODO?

diff --git a/hibernate-search-orm/src/test/java/org/hibernate/search/test/interceptor/Blog.java b/hibernate-search-orm/src/test/java/org/hibernate/search/test/interceptor/Blog.java
@@ -44,7 +44,7 @@ public class Blog {
 	public void setTitle(String title) {  this.title = title; }
 	private String title;
 
-	@Field
+	//state used by interceptor but not indexed on purpose to test optimization disablement
 	public BlogStatus getStatus() { return status; }
 	public void setStatus(BlogStatus status) {  this.status = status; }
 	private BlogStatus status;