Author: navssurtani Date: 2008-06-30 13:25:51 -0400 (Mon, 30 Jun 2008) New Revision: 14831 Modified: search/branches/jboss_cache_integration/pom.xml search/branches/jboss_cache_integration/src/java/org/hibernate/search/engine/DocumentBuilder.java Log: Corrected version Modified: search/branches/jboss_cache_integration/pom.xml =================================================================== --- search/branches/jboss_cache_integration/pom.xml 2008-06-30 17:14:47 UTC (rev 14830) +++ search/branches/jboss_cache_integration/pom.xml 2008-06-30 17:25:51 UTC (rev 14831) @@ -4,7 +4,7 @@ <modelVersion>4.0.0</modelVersion> <groupId>org.hibernate.sandbox</groupId> <artifactId>hibernate-search-gsoc</artifactId> - <version>3.1.0-SNAPSHOT</version> + <version>3.1.1-SNAPSHOT</version> <description>Hibernate Search - JBoss Cache Integration Branch</description> <dependencies> <dependency> Modified: search/branches/jboss_cache_integration/src/java/org/hibernate/search/engine/DocumentBuilder.java =================================================================== --- search/branches/jboss_cache_integration/src/java/org/hibernate/search/engine/DocumentBuilder.java 2008-06-30 17:14:47 UTC (rev 14830) +++ search/branches/jboss_cache_integration/src/java/org/hibernate/search/engine/DocumentBuilder.java 2008-06-30 17:25:51 UTC (rev 14831) @@ -54,796 +54,936 @@ * @author Richard Hallier * @author Hardy Ferentschik */ -public class DocumentBuilder<T> { - private static final Logger log = LoggerFactory.getLogger( DocumentBuilder.class ); +public class DocumentBuilder<T> +{ + private static final Logger log = LoggerFactory.getLogger(DocumentBuilder.class); - private final PropertiesMetadata rootPropertiesMetadata = new PropertiesMetadata(); - private final XClass beanClass; - private final DirectoryProvider[] directoryProviders; - private final IndexShardingStrategy shardingStrategy; - private String idKeywordName; - private XMember idGetter; - private Float idBoost; - public static final String CLASS_FIELDNAME = "_hibernate_class"; - private TwoWayFieldBridge idBridge; - private Set<Class> mappedSubclasses = new HashSet<Class>(); - private ReflectionManager reflectionManager; - private int level = 0; - private int maxLevel = Integer.MAX_VALUE; - private final ScopedAnalyzer analyzer = new ScopedAnalyzer(); - private Similarity similarity; - private boolean isRoot; - //if composite id, use of (a, b) in ((1,2), (3,4)) fails on most database - private boolean safeFromTupleId; + private final PropertiesMetadata rootPropertiesMetadata = new PropertiesMetadata(); + private final XClass beanClass; + private final DirectoryProvider[] directoryProviders; + private final IndexShardingStrategy shardingStrategy; + private String idKeywordName; + private XMember idGetter; + private Float idBoost; + public static final String CLASS_FIELDNAME = "_hibernate_class"; + private TwoWayFieldBridge idBridge; + private Set<Class> mappedSubclasses = new HashSet<Class>(); + private ReflectionManager reflectionManager; + private int level = 0; + private int maxLevel = Integer.MAX_VALUE; + private final ScopedAnalyzer analyzer = new ScopedAnalyzer(); + private Similarity similarity; + private boolean isRoot; + //if composite id, use of (a, b) in ((1,2), (3,4)) fails on most database + private boolean safeFromTupleId; - public boolean isRoot() { - return isRoot; - } + public boolean isRoot() + { + return isRoot; + } - public DocumentBuilder(XClass clazz, InitContext context, DirectoryProvider[] directoryProviders, - IndexShardingStrategy shardingStrategy, ReflectionManager reflectionManager) { - this.beanClass = clazz; - this.directoryProviders = directoryProviders; - this.shardingStrategy = shardingStrategy; - //FIXME get rid of it when boost is stored? - this.reflectionManager = reflectionManager; - this.similarity = context.getDefaultSimilarity(); + public DocumentBuilder(XClass clazz, InitContext context, DirectoryProvider[] directoryProviders, + IndexShardingStrategy shardingStrategy, ReflectionManager reflectionManager) + { + this.beanClass = clazz; + this.directoryProviders = directoryProviders; + this.shardingStrategy = shardingStrategy; + //FIXME get rid of it when boost is stored? + this.reflectionManager = reflectionManager; + this.similarity = context.getDefaultSimilarity(); - if ( clazz == null ) throw new AssertionFailure( "Unable to build a DocumentBuilder with a null class" ); - rootPropertiesMetadata.boost = getBoost( clazz ); - rootPropertiesMetadata.analyzer = context.getDefaultAnalyzer(); - Set<XClass> processedClasses = new HashSet<XClass>(); - processedClasses.add( clazz ); - initializeMembers( clazz, rootPropertiesMetadata, true, "", processedClasses, context ); - //processedClasses.remove( clazz ); for the sake of completness - this.analyzer.setGlobalAnalyzer( rootPropertiesMetadata.analyzer ); - if ( idKeywordName == null ) { + if (clazz == null) throw new AssertionFailure("Unable to build a DocumentBuilder with a null class"); + rootPropertiesMetadata.boost = getBoost(clazz); + rootPropertiesMetadata.analyzer = context.getDefaultAnalyzer(); + Set<XClass> processedClasses = new HashSet<XClass>(); + processedClasses.add(clazz); + initializeMembers(clazz, rootPropertiesMetadata, true, "", processedClasses, context); + //processedClasses.remove( clazz ); for the sake of completness + this.analyzer.setGlobalAnalyzer(rootPropertiesMetadata.analyzer); + if (idKeywordName == null) + { // if no DocumentId then check if we have a ProvidedId instead - if (clazz.getAnnotation(ProvidedId.class) == null) { - throw new SearchException( "No document id in: " + clazz.getName() ); + if (clazz.getAnnotation(ProvidedId.class) == null) + { + throw new SearchException("No document id in: " + clazz.getName()); } // else { // // DON'T throw an exception. This is OK. // } - } - //if composite id, use of (a, b) in ((1,2)TwoWayString2FieldBridgeAdaptor, (3,4)) fails on most database - //a TwoWayString2FieldBridgeAdaptor is never a composite id - safeFromTupleId = TwoWayString2FieldBridgeAdaptor.class.isAssignableFrom( idBridge.getClass() ); - } + } + //if composite id, use of (a, b) in ((1,2)TwoWayString2FieldBridgeAdaptor, (3,4)) fails on most database + //a TwoWayString2FieldBridgeAdaptor is never a composite id + safeFromTupleId = TwoWayString2FieldBridgeAdaptor.class.isAssignableFrom(idBridge.getClass()); + } - private Analyzer getAnalyzer(XAnnotatedElement annotatedElement, InitContext context) { - org.hibernate.search.annotations.Analyzer analyzerAnn = - annotatedElement.getAnnotation( org.hibernate.search.annotations.Analyzer.class ); - return getAnalyzer( analyzerAnn, context ); - } + private Analyzer getAnalyzer(XAnnotatedElement annotatedElement, InitContext context) + { + org.hibernate.search.annotations.Analyzer analyzerAnn = + annotatedElement.getAnnotation(org.hibernate.search.annotations.Analyzer.class); + return getAnalyzer(analyzerAnn, context); + } - private Analyzer getAnalyzer(org.hibernate.search.annotations.Analyzer analyzerAnn, InitContext context) { - Class analyzerClass = analyzerAnn == null ? void.class : analyzerAnn.impl(); - if ( analyzerClass == void.class ) { - String definition = analyzerAnn == null ? "" : analyzerAnn.definition(); - if ( StringHelper.isEmpty( definition ) ) { - return null; - } - else { + private Analyzer getAnalyzer(org.hibernate.search.annotations.Analyzer analyzerAnn, InitContext context) + { + Class analyzerClass = analyzerAnn == null ? void.class : analyzerAnn.impl(); + if (analyzerClass == void.class) + { + String definition = analyzerAnn == null ? "" : analyzerAnn.definition(); + if (StringHelper.isEmpty(definition)) + { + return null; + } + else + { - return context.buildLazyAnalyzer( definition ); - } - } - else { - try { - return (Analyzer) analyzerClass.newInstance(); - } - catch (ClassCastException e) { - throw new SearchException( - "Lucene analyzer does not implement " + Analyzer.class.getName() + ": " + analyzerClass.getName(), e - ); - } - catch (Exception e) { - throw new SearchException( "Failed to instantiate lucene analyzer with type " + analyzerClass.getName(), e ); - } - } - } + return context.buildLazyAnalyzer(definition); + } + } + else + { + try + { + return (Analyzer) analyzerClass.newInstance(); + } + catch (ClassCastException e) + { + throw new SearchException( + "Lucene analyzer does not implement " + Analyzer.class.getName() + ": " + analyzerClass.getName(), e + ); + } + catch (Exception e) + { + throw new SearchException("Failed to instantiate lucene analyzer with type " + analyzerClass.getName(), e); + } + } + } - private void initializeMembers(XClass clazz, PropertiesMetadata propertiesMetadata, boolean isRoot, String prefix, - Set<XClass> processedClasses, InitContext context) { - List<XClass> hierarchy = new ArrayList<XClass>(); - for (XClass currClass = clazz; currClass != null; currClass = currClass.getSuperclass()) { - hierarchy.add( currClass ); - } - Class similarityClass = null; - for (int index = hierarchy.size() - 1; index >= 0; index--) { - XClass currClass = hierarchy.get( index ); - /** - * Override the default analyzer for the properties if the class hold one - * That's the reason we go down the hierarchy - */ - Analyzer analyzer = getAnalyzer( currClass, context ); + private void initializeMembers(XClass clazz, PropertiesMetadata propertiesMetadata, boolean isRoot, String prefix, + Set<XClass> processedClasses, InitContext context) + { + List<XClass> hierarchy = new ArrayList<XClass>(); + for (XClass currClass = clazz; currClass != null; currClass = currClass.getSuperclass()) + { + hierarchy.add(currClass); + } + Class similarityClass = null; + for (int index = hierarchy.size() - 1; index >= 0; index--) + { + XClass currClass = hierarchy.get(index); + /** + * Override the default analyzer for the properties if the class hold one + * That's the reason we go down the hierarchy + */ + Analyzer analyzer = getAnalyzer(currClass, context); - if ( analyzer != null ) { - propertiesMetadata.analyzer = analyzer; - } - getAnalyzerDefs(currClass, context); - // Check for any ClassBridges annotation. - ClassBridges classBridgesAnn = currClass.getAnnotation( ClassBridges.class ); - if ( classBridgesAnn != null ) { - ClassBridge[] cbs = classBridgesAnn.value(); - for (ClassBridge cb : cbs) { - bindClassAnnotation( prefix, propertiesMetadata, cb, context ); - } - } + if (analyzer != null) + { + propertiesMetadata.analyzer = analyzer; + } + getAnalyzerDefs(currClass, context); + // Check for any ClassBridges annotation. + ClassBridges classBridgesAnn = currClass.getAnnotation(ClassBridges.class); + if (classBridgesAnn != null) + { + ClassBridge[] cbs = classBridgesAnn.value(); + for (ClassBridge cb : cbs) + { + bindClassAnnotation(prefix, propertiesMetadata, cb, context); + } + } - // Check for any ClassBridge style of annotations. - ClassBridge classBridgeAnn = currClass.getAnnotation( ClassBridge.class ); - if ( classBridgeAnn != null ) { - bindClassAnnotation( prefix, propertiesMetadata, classBridgeAnn, context ); - } + // Check for any ClassBridge style of annotations. + ClassBridge classBridgeAnn = currClass.getAnnotation(ClassBridge.class); + if (classBridgeAnn != null) + { + bindClassAnnotation(prefix, propertiesMetadata, classBridgeAnn, context); + } - //Get similarity - //TODO: similarity form @IndexedEmbedded are not taken care of. Exception?? - if ( isRoot ) { - org.hibernate.search.annotations.Similarity similarityAnn = currClass.getAnnotation( org.hibernate.search.annotations.Similarity.class ); - if ( similarityAnn != null ) { - if ( similarityClass != null ) { - throw new SearchException( "Multiple Similarities defined in the same class hierarchy: " + beanClass.getName() ); - } - similarityClass = similarityAnn.impl(); - } - } + //Get similarity + //TODO: similarity form @IndexedEmbedded are not taken care of. Exception?? + if (isRoot) + { + org.hibernate.search.annotations.Similarity similarityAnn = currClass.getAnnotation(org.hibernate.search.annotations.Similarity.class); + if (similarityAnn != null) + { + if (similarityClass != null) + { + throw new SearchException("Multiple Similarities defined in the same class hierarchy: " + beanClass.getName()); + } + similarityClass = similarityAnn.impl(); + } + } - //rejecting non properties (ie regular methods) because the object is loaded from Hibernate, - // so indexing a non property does not make sense - List<XProperty> methods = currClass.getDeclaredProperties( XClass.ACCESS_PROPERTY ); - for (XProperty method : methods) { - initializeMember( method, propertiesMetadata, isRoot, prefix, processedClasses, context ); - } + //rejecting non properties (ie regular methods) because the object is loaded from Hibernate, + // so indexing a non property does not make sense + List<XProperty> methods = currClass.getDeclaredProperties(XClass.ACCESS_PROPERTY); + for (XProperty method : methods) + { + initializeMember(method, propertiesMetadata, isRoot, prefix, processedClasses, context); + } - List<XProperty> fields = currClass.getDeclaredProperties( XClass.ACCESS_FIELD ); - for (XProperty field : fields) { - initializeMember( field, propertiesMetadata, isRoot, prefix, processedClasses, context ); - } - } - if ( isRoot && similarityClass != null ) { - try { - similarity = (Similarity) similarityClass.newInstance(); - } - catch (Exception e) { - log.error( "Exception attempting to instantiate Similarity '{}' set for {}", - similarityClass.getName(), beanClass.getName() ); - } - } - } + List<XProperty> fields = currClass.getDeclaredProperties(XClass.ACCESS_FIELD); + for (XProperty field : fields) + { + initializeMember(field, propertiesMetadata, isRoot, prefix, processedClasses, context); + } + } + if (isRoot && similarityClass != null) + { + try + { + similarity = (Similarity) similarityClass.newInstance(); + } + catch (Exception e) + { + log.error("Exception attempting to instantiate Similarity '{}' set for {}", + similarityClass.getName(), beanClass.getName()); + } + } + } - private void getAnalyzerDefs(XAnnotatedElement annotatedElement, InitContext context) { - AnalyzerDefs defs = annotatedElement.getAnnotation( AnalyzerDefs.class ); - if ( defs != null ) { - for (AnalyzerDef def : defs.value()) { - context.addAnalyzerDef( def ); - } - } - AnalyzerDef def = annotatedElement.getAnnotation( AnalyzerDef.class ); - context.addAnalyzerDef( def ); - } + private void getAnalyzerDefs(XAnnotatedElement annotatedElement, InitContext context) + { + AnalyzerDefs defs = annotatedElement.getAnnotation(AnalyzerDefs.class); + if (defs != null) + { + for (AnalyzerDef def : defs.value()) + { + context.addAnalyzerDef(def); + } + } + AnalyzerDef def = annotatedElement.getAnnotation(AnalyzerDef.class); + context.addAnalyzerDef(def); + } - public String getIdentifierName() { - return idGetter.getName(); - } + public String getIdentifierName() + { + return idGetter.getName(); + } - public Similarity getSimilarity() { - return similarity; - } + public Similarity getSimilarity() + { + return similarity; + } - private void initializeMember(XProperty member, PropertiesMetadata propertiesMetadata, boolean isRoot, - String prefix, Set<XClass> processedClasses, InitContext context) { + private void initializeMember(XProperty member, PropertiesMetadata propertiesMetadata, boolean isRoot, + String prefix, Set<XClass> processedClasses, InitContext context) + { - DocumentId documentIdAnn = member.getAnnotation( DocumentId.class ); - if ( documentIdAnn != null ) { - if ( isRoot ) { - if ( idKeywordName != null ) { - throw new AssertionFailure( "Two document id assigned: " - + idKeywordName + " and " + BinderHelper.getAttributeName( member, documentIdAnn.name() ) ); - } - idKeywordName = prefix + BinderHelper.getAttributeName( member, documentIdAnn.name() ); - FieldBridge fieldBridge = BridgeFactory.guessType( null, member, reflectionManager ); - if ( fieldBridge instanceof TwoWayFieldBridge ) { - idBridge = (TwoWayFieldBridge) fieldBridge; - } - else { - throw new SearchException( - "Bridge for document id does not implement TwoWayFieldBridge: " + member.getName() ); - } - idBoost = getBoost( member ); - setAccessible( member ); - idGetter = member; - } - else { - //component should index their document id - setAccessible( member ); - propertiesMetadata.fieldGetters.add( member ); - String fieldName = prefix + BinderHelper.getAttributeName( member, documentIdAnn.name() ); - propertiesMetadata.fieldNames.add( fieldName ); - propertiesMetadata.fieldStore.add( getStore( Store.YES ) ); - propertiesMetadata.fieldIndex.add( getIndex( Index.UN_TOKENIZED ) ); - propertiesMetadata.fieldTermVectors.add( getTermVector( TermVector.NO ) ); - propertiesMetadata.fieldBridges.add( BridgeFactory.guessType( null, member, reflectionManager ) ); - // property > entity analyzer (no field analyzer) - Analyzer analyzer = getAnalyzer( member, context ); - if ( analyzer == null ) analyzer = propertiesMetadata.analyzer; - if ( analyzer == null ) throw new AssertionFailure( "Analizer should not be undefined" ); - this.analyzer.addScopedAnalyzer( fieldName, analyzer ); - } - } - { - org.hibernate.search.annotations.Field fieldAnn = - member.getAnnotation( org.hibernate.search.annotations.Field.class ); - if ( fieldAnn != null ) { - bindFieldAnnotation( member, propertiesMetadata, prefix, fieldAnn, context ); - } - } - { - org.hibernate.search.annotations.Fields fieldsAnn = - member.getAnnotation( org.hibernate.search.annotations.Fields.class ); - if ( fieldsAnn != null ) { - for (org.hibernate.search.annotations.Field fieldAnn : fieldsAnn.value()) { - bindFieldAnnotation( member, propertiesMetadata, prefix, fieldAnn, context ); - } - } - } - getAnalyzerDefs( member, context ); + DocumentId documentIdAnn = member.getAnnotation(DocumentId.class); + if (documentIdAnn != null) + { + if (isRoot) + { + if (idKeywordName != null) + { + throw new AssertionFailure("Two document id assigned: " + + idKeywordName + " and " + BinderHelper.getAttributeName(member, documentIdAnn.name())); + } + idKeywordName = prefix + BinderHelper.getAttributeName(member, documentIdAnn.name()); + FieldBridge fieldBridge = BridgeFactory.guessType(null, member, reflectionManager); + if (fieldBridge instanceof TwoWayFieldBridge) + { + idBridge = (TwoWayFieldBridge) fieldBridge; + } + else + { + throw new SearchException( + "Bridge for document id does not implement TwoWayFieldBridge: " + member.getName()); + } + idBoost = getBoost(member); + setAccessible(member); + idGetter = member; + } + else + { + //component should index their document id + setAccessible(member); + propertiesMetadata.fieldGetters.add(member); + String fieldName = prefix + BinderHelper.getAttributeName(member, documentIdAnn.name()); + propertiesMetadata.fieldNames.add(fieldName); + propertiesMetadata.fieldStore.add(getStore(Store.YES)); + propertiesMetadata.fieldIndex.add(getIndex(Index.UN_TOKENIZED)); + propertiesMetadata.fieldTermVectors.add(getTermVector(TermVector.NO)); + propertiesMetadata.fieldBridges.add(BridgeFactory.guessType(null, member, reflectionManager)); + // property > entity analyzer (no field analyzer) + Analyzer analyzer = getAnalyzer(member, context); + if (analyzer == null) analyzer = propertiesMetadata.analyzer; + if (analyzer == null) throw new AssertionFailure("Analizer should not be undefined"); + this.analyzer.addScopedAnalyzer(fieldName, analyzer); + } + } + { + org.hibernate.search.annotations.Field fieldAnn = + member.getAnnotation(org.hibernate.search.annotations.Field.class); + if (fieldAnn != null) + { + bindFieldAnnotation(member, propertiesMetadata, prefix, fieldAnn, context); + } + } + { + org.hibernate.search.annotations.Fields fieldsAnn = + member.getAnnotation(org.hibernate.search.annotations.Fields.class); + if (fieldsAnn != null) + { + for (org.hibernate.search.annotations.Field fieldAnn : fieldsAnn.value()) + { + bindFieldAnnotation(member, propertiesMetadata, prefix, fieldAnn, context); + } + } + } + getAnalyzerDefs(member, context); - IndexedEmbedded embeddedAnn = member.getAnnotation( IndexedEmbedded.class ); - if ( embeddedAnn != null ) { - int oldMaxLevel = maxLevel; - int potentialLevel = embeddedAnn.depth() + level; - if ( potentialLevel < 0 ) { - potentialLevel = Integer.MAX_VALUE; - } - maxLevel = potentialLevel > maxLevel ? maxLevel : potentialLevel; - level++; + IndexedEmbedded embeddedAnn = member.getAnnotation(IndexedEmbedded.class); + if (embeddedAnn != null) + { + int oldMaxLevel = maxLevel; + int potentialLevel = embeddedAnn.depth() + level; + if (potentialLevel < 0) + { + potentialLevel = Integer.MAX_VALUE; + } + maxLevel = potentialLevel > maxLevel ? maxLevel : potentialLevel; + level++; - XClass elementClass; - if ( void.class == embeddedAnn.targetElement() ) { - elementClass = member.getElementClass(); - } - else { - elementClass = reflectionManager.toXClass( embeddedAnn.targetElement() ); - } - if ( maxLevel == Integer.MAX_VALUE //infinite - && processedClasses.contains( elementClass ) ) { - throw new SearchException( - "Circular reference. Duplicate use of " - + elementClass.getName() - + " in root entity " + beanClass.getName() - + "#" + buildEmbeddedPrefix( prefix, embeddedAnn, member ) - ); - } - if ( level <= maxLevel ) { - processedClasses.add( elementClass ); //push + XClass elementClass; + if (void.class == embeddedAnn.targetElement()) + { + elementClass = member.getElementClass(); + } + else + { + elementClass = reflectionManager.toXClass(embeddedAnn.targetElement()); + } + if (maxLevel == Integer.MAX_VALUE //infinite + && processedClasses.contains(elementClass)) + { + throw new SearchException( + "Circular reference. Duplicate use of " + + elementClass.getName() + + " in root entity " + beanClass.getName() + + "#" + buildEmbeddedPrefix(prefix, embeddedAnn, member) + ); + } + if (level <= maxLevel) + { + processedClasses.add(elementClass); //push - setAccessible( member ); - propertiesMetadata.embeddedGetters.add( member ); - PropertiesMetadata metadata = new PropertiesMetadata(); - propertiesMetadata.embeddedPropertiesMetadata.add( metadata ); - metadata.boost = getBoost( member ); - //property > entity analyzer - Analyzer analyzer = getAnalyzer( member, context ); - metadata.analyzer = analyzer != null ? analyzer : propertiesMetadata.analyzer; - String localPrefix = buildEmbeddedPrefix( prefix, embeddedAnn, member ); - initializeMembers( elementClass, metadata, false, localPrefix, processedClasses, context ); - /** - * We will only index the "expected" type but that's OK, HQL cannot do downcasting either - */ - if ( member.isArray() ) { - propertiesMetadata.embeddedContainers.add( PropertiesMetadata.Container.ARRAY ); - } - else if ( member.isCollection() ) { - if ( Map.class.equals( member.getCollectionClass() ) ) { - //hum subclasses etc etc?? - propertiesMetadata.embeddedContainers.add( PropertiesMetadata.Container.MAP ); - } - else { - propertiesMetadata.embeddedContainers.add( PropertiesMetadata.Container.COLLECTION ); - } - } - else { - propertiesMetadata.embeddedContainers.add( PropertiesMetadata.Container.OBJECT ); - } + setAccessible(member); + propertiesMetadata.embeddedGetters.add(member); + PropertiesMetadata metadata = new PropertiesMetadata(); + propertiesMetadata.embeddedPropertiesMetadata.add(metadata); + metadata.boost = getBoost(member); + //property > entity analyzer + Analyzer analyzer = getAnalyzer(member, context); + metadata.analyzer = analyzer != null ? analyzer : propertiesMetadata.analyzer; + String localPrefix = buildEmbeddedPrefix(prefix, embeddedAnn, member); + initializeMembers(elementClass, metadata, false, localPrefix, processedClasses, context); + /** + * We will only index the "expected" type but that's OK, HQL cannot do downcasting either + */ + if (member.isArray()) + { + propertiesMetadata.embeddedContainers.add(PropertiesMetadata.Container.ARRAY); + } + else if (member.isCollection()) + { + if (Map.class.equals(member.getCollectionClass())) + { + //hum subclasses etc etc?? + propertiesMetadata.embeddedContainers.add(PropertiesMetadata.Container.MAP); + } + else + { + propertiesMetadata.embeddedContainers.add(PropertiesMetadata.Container.COLLECTION); + } + } + else + { + propertiesMetadata.embeddedContainers.add(PropertiesMetadata.Container.OBJECT); + } - processedClasses.remove( elementClass ); //pop - } - else if ( log.isTraceEnabled() ) { - String localPrefix = buildEmbeddedPrefix( prefix, embeddedAnn, member ); - log.trace( "depth reached, ignoring {}", localPrefix ); - } + processedClasses.remove(elementClass); //pop + } + else if (log.isTraceEnabled()) + { + String localPrefix = buildEmbeddedPrefix(prefix, embeddedAnn, member); + log.trace("depth reached, ignoring {}", localPrefix); + } - level--; - maxLevel = oldMaxLevel; //set back the the old max level - } + level--; + maxLevel = oldMaxLevel; //set back the the old max level + } - ContainedIn containedAnn = member.getAnnotation( ContainedIn.class ); - if ( containedAnn != null ) { - setAccessible( member ); - propertiesMetadata.containedInGetters.add( member ); - } - } + ContainedIn containedAnn = member.getAnnotation(ContainedIn.class); + if (containedAnn != null) + { + setAccessible(member); + propertiesMetadata.containedInGetters.add(member); + } + } - private void bindClassAnnotation(String prefix, PropertiesMetadata propertiesMetadata, ClassBridge ann, InitContext context) { - //FIXME name should be prefixed - String fieldName = prefix + ann.name(); - propertiesMetadata.classNames.add( fieldName ); - propertiesMetadata.classStores.add( getStore( ann.store() ) ); - propertiesMetadata.classIndexes.add( getIndex( ann.index() ) ); - propertiesMetadata.classTermVectors.add( getTermVector( ann.termVector() ) ); - propertiesMetadata.classBridges.add( BridgeFactory.extractType( ann ) ); - propertiesMetadata.classBoosts.add( ann.boost().value() ); + private void bindClassAnnotation(String prefix, PropertiesMetadata propertiesMetadata, ClassBridge ann, InitContext context) + { + //FIXME name should be prefixed + String fieldName = prefix + ann.name(); + propertiesMetadata.classNames.add(fieldName); + propertiesMetadata.classStores.add(getStore(ann.store())); + propertiesMetadata.classIndexes.add(getIndex(ann.index())); + propertiesMetadata.classTermVectors.add(getTermVector(ann.termVector())); + propertiesMetadata.classBridges.add(BridgeFactory.extractType(ann)); + propertiesMetadata.classBoosts.add(ann.boost().value()); - Analyzer analyzer = getAnalyzer( ann.analyzer(), context ); - if ( analyzer == null ) analyzer = propertiesMetadata.analyzer; - if ( analyzer == null ) throw new AssertionFailure( "Analyzer should not be undefined" ); - this.analyzer.addScopedAnalyzer( fieldName, analyzer ); - } + Analyzer analyzer = getAnalyzer(ann.analyzer(), context); + if (analyzer == null) analyzer = propertiesMetadata.analyzer; + if (analyzer == null) throw new AssertionFailure("Analyzer should not be undefined"); + this.analyzer.addScopedAnalyzer(fieldName, analyzer); + } - private void bindFieldAnnotation(XProperty member, PropertiesMetadata propertiesMetadata, String prefix, org.hibernate.search.annotations.Field fieldAnn, InitContext context) { - setAccessible( member ); - propertiesMetadata.fieldGetters.add( member ); - String fieldName = prefix + BinderHelper.getAttributeName( member, fieldAnn.name() ); - propertiesMetadata.fieldNames.add( fieldName ); - propertiesMetadata.fieldStore.add( getStore( fieldAnn.store() ) ); - propertiesMetadata.fieldIndex.add( getIndex( fieldAnn.index() ) ); - propertiesMetadata.fieldTermVectors.add( getTermVector( fieldAnn.termVector() ) ); - propertiesMetadata.fieldBridges.add( BridgeFactory.guessType( fieldAnn, member, reflectionManager ) ); + private void bindFieldAnnotation(XProperty member, PropertiesMetadata propertiesMetadata, String prefix, org.hibernate.search.annotations.Field fieldAnn, InitContext context) + { + setAccessible(member); + propertiesMetadata.fieldGetters.add(member); + String fieldName = prefix + BinderHelper.getAttributeName(member, fieldAnn.name()); + propertiesMetadata.fieldNames.add(fieldName); + propertiesMetadata.fieldStore.add(getStore(fieldAnn.store())); + propertiesMetadata.fieldIndex.add(getIndex(fieldAnn.index())); + propertiesMetadata.fieldTermVectors.add(getTermVector(fieldAnn.termVector())); + propertiesMetadata.fieldBridges.add(BridgeFactory.guessType(fieldAnn, member, reflectionManager)); - // Field > property > entity analyzer - Analyzer analyzer = getAnalyzer( fieldAnn.analyzer(), context ); - if ( analyzer == null ) analyzer = getAnalyzer( member, context ); - if ( analyzer == null ) analyzer = propertiesMetadata.analyzer; - if ( analyzer == null ) throw new AssertionFailure( "Analizer should not be undefined" ); - this.analyzer.addScopedAnalyzer( fieldName, analyzer ); - } + // Field > property > entity analyzer + Analyzer analyzer = getAnalyzer(fieldAnn.analyzer(), context); + if (analyzer == null) analyzer = getAnalyzer(member, context); + if (analyzer == null) analyzer = propertiesMetadata.analyzer; + if (analyzer == null) throw new AssertionFailure("Analizer should not be undefined"); + this.analyzer.addScopedAnalyzer(fieldName, analyzer); + } - private String buildEmbeddedPrefix(String prefix, IndexedEmbedded embeddedAnn, XProperty member) { - String localPrefix = prefix; - if ( ".".equals( embeddedAnn.prefix() ) ) { - //default to property name - localPrefix += member.getName() + '.'; - } - else { - localPrefix += embeddedAnn.prefix(); - } - return localPrefix; - } + private String buildEmbeddedPrefix(String prefix, IndexedEmbedded embeddedAnn, XProperty member) + { + String localPrefix = prefix; + if (".".equals(embeddedAnn.prefix())) + { + //default to property name + localPrefix += member.getName() + '.'; + } + else + { + localPrefix += embeddedAnn.prefix(); + } + return localPrefix; + } - private Field.Store getStore(Store store) { - switch ( store ) { - case NO: - return Field.Store.NO; - case YES: - return Field.Store.YES; - case COMPRESS: - return Field.Store.COMPRESS; - default: - throw new AssertionFailure( "Unexpected Store: " + store ); - } - } + private Field.Store getStore(Store store) + { + switch (store) + { + case NO: + return Field.Store.NO; + case YES: + return Field.Store.YES; + case COMPRESS: + return Field.Store.COMPRESS; + default: + throw new AssertionFailure("Unexpected Store: " + store); + } + } - private Field.TermVector getTermVector(TermVector vector) { - switch ( vector ) { - case NO: - return Field.TermVector.NO; - case YES: - return Field.TermVector.YES; - case WITH_OFFSETS: - return Field.TermVector.WITH_OFFSETS; - case WITH_POSITIONS: - return Field.TermVector.WITH_POSITIONS; - case WITH_POSITION_OFFSETS: - return Field.TermVector.WITH_POSITIONS_OFFSETS; - default: - throw new AssertionFailure( "Unexpected TermVector: " + vector ); - } - } + private Field.TermVector getTermVector(TermVector vector) + { + switch (vector) + { + case NO: + return Field.TermVector.NO; + case YES: + return Field.TermVector.YES; + case WITH_OFFSETS: + return Field.TermVector.WITH_OFFSETS; + case WITH_POSITIONS: + return Field.TermVector.WITH_POSITIONS; + case WITH_POSITION_OFFSETS: + return Field.TermVector.WITH_POSITIONS_OFFSETS; + default: + throw new AssertionFailure("Unexpected TermVector: " + vector); + } + } - private Field.Index getIndex(Index index) { - switch ( index ) { - case NO: - return Field.Index.NO; - case NO_NORMS: - return Field.Index.NO_NORMS; - case TOKENIZED: - return Field.Index.TOKENIZED; - case UN_TOKENIZED: - return Field.Index.UN_TOKENIZED; - default: - throw new AssertionFailure( "Unexpected Index: " + index ); - } - } + private Field.Index getIndex(Index index) + { + switch (index) + { + case NO: + return Field.Index.NO; + case NO_NORMS: + return Field.Index.NO_NORMS; + case TOKENIZED: + return Field.Index.TOKENIZED; + case UN_TOKENIZED: + return Field.Index.UN_TOKENIZED; + default: + throw new AssertionFailure("Unexpected Index: " + index); + } + } - private Float getBoost(XAnnotatedElement element) { - if ( element == null ) return null; - Boost boost = element.getAnnotation( Boost.class ); - return boost != null ? - boost.value() : - null; - } + private Float getBoost(XAnnotatedElement element) + { + if (element == null) return null; + Boost boost = element.getAnnotation(Boost.class); + return boost != null ? + boost.value() : + null; + } - private Object getMemberValue(Object bean, XMember getter) { - Object value; - try { - value = getter.invoke( bean ); - } - catch (Exception e) { - throw new IllegalStateException( "Could not get property value", e ); - } - return value; - } + private Object getMemberValue(Object bean, XMember getter) + { + Object value; + try + { + value = getter.invoke(bean); + } + catch (Exception e) + { + throw new IllegalStateException("Could not get property value", e); + } + return value; + } - //TODO could we use T instead of EntityClass? - public void addWorkToQueue(Class entityClass, T entity, Serializable id, WorkType workType, List<LuceneWork> queue, SearchFactoryImplementor searchFactoryImplementor) { - //TODO with the caller loop we are in a n^2: optimize it using a HashMap for work recognition - for (LuceneWork luceneWork : queue) { - //any work on the same entity should be ignored - if ( luceneWork.getEntityClass() == entityClass - ) { - Serializable currentId = luceneWork.getId(); - if ( currentId != null && currentId.equals( id ) ) { //find a way to use Type.equals(x,y) - return; - } - //TODO do something to avoid multiple PURGE ALL and OPTIMIZE - } + //TODO could we use T instead of EntityClass? + public void addWorkToQueue(Class entityClass, T entity, Serializable id, WorkType workType, List<LuceneWork> queue, SearchFactoryImplementor searchFactoryImplementor) + { + //TODO with the caller loop we are in a n^2: optimize it using a HashMap for work recognition + for (LuceneWork luceneWork : queue) + { + //any work on the same entity should be ignored + if (luceneWork.getEntityClass() == entityClass + ) + { + Serializable currentId = luceneWork.getId(); + if (currentId != null && currentId.equals(id)) + { //find a way to use Type.equals(x,y) + return; + } + //TODO do something to avoid multiple PURGE ALL and OPTIMIZE + } - } - boolean searchForContainers = false; - String idInString = idBridge.objectToString( id ); - if ( workType == WorkType.ADD ) { - Document doc = getDocument( entity, id ); - queue.add( new AddLuceneWork( id, idInString, entityClass, doc ) ); - searchForContainers = true; - } - else if ( workType == WorkType.DELETE || workType == WorkType.PURGE ) { - queue.add( new DeleteLuceneWork( id, idInString, entityClass ) ); - } - else if ( workType == WorkType.PURGE_ALL ) { - queue.add( new PurgeAllLuceneWork( entityClass ) ); - } - else if ( workType == WorkType.UPDATE || workType == WorkType.COLLECTION ) { - Document doc = getDocument( entity, id ); - /** - * even with Lucene 2.1, use of indexWriter to update is not an option - * We can only delete by term, and the index doesn't have a term that - * uniquely identify the entry. - * But essentially the optimization we are doing is the same Lucene is doing, the only extra cost is the - * double file opening. - */ - queue.add( new DeleteLuceneWork( id, idInString, entityClass ) ); - queue.add( new AddLuceneWork( id, idInString, entityClass, doc ) ); - searchForContainers = true; - } - else if ( workType == WorkType.INDEX ) { - Document doc = getDocument( entity, id ); - queue.add( new DeleteLuceneWork( id, idInString, entityClass ) ); - LuceneWork work = new AddLuceneWork( id, idInString, entityClass, doc ); - work.setBatch( true ); - queue.add( work ); - searchForContainers = true; - } + } + boolean searchForContainers = false; + String idInString = idBridge.objectToString(id); + if (workType == WorkType.ADD) + { + Document doc = getDocument(entity, id); + queue.add(new AddLuceneWork(id, idInString, entityClass, doc)); + searchForContainers = true; + } + else if (workType == WorkType.DELETE || workType == WorkType.PURGE) + { + queue.add(new DeleteLuceneWork(id, idInString, entityClass)); + } + else if (workType == WorkType.PURGE_ALL) + { + queue.add(new PurgeAllLuceneWork(entityClass)); + } + else if (workType == WorkType.UPDATE || workType == WorkType.COLLECTION) + { + Document doc = getDocument(entity, id); + /** + * even with Lucene 2.1, use of indexWriter to update is not an option + * We can only delete by term, and the index doesn't have a term that + * uniquely identify the entry. + * But essentially the optimization we are doing is the same Lucene is doing, the only extra cost is the + * double file opening. + */ + queue.add(new DeleteLuceneWork(id, idInString, entityClass)); + queue.add(new AddLuceneWork(id, idInString, entityClass, doc)); + searchForContainers = true; + } + else if (workType == WorkType.INDEX) + { + Document doc = getDocument(entity, id); + queue.add(new DeleteLuceneWork(id, idInString, entityClass)); + LuceneWork work = new AddLuceneWork(id, idInString, entityClass, doc); + work.setBatch(true); + queue.add(work); + searchForContainers = true; + } - else { - throw new AssertionFailure( "Unknown WorkType: " + workType ); - } + else + { + throw new AssertionFailure("Unknown WorkType: " + workType); + } - /** - * When references are changed, either null or another one, we expect dirty checking to be triggered (both sides - * have to be updated) - * When the internal object is changed, we apply the {Add|Update}Work on containedIns - */ - if ( searchForContainers ) { - processContainedIn( entity, queue, rootPropertiesMetadata, searchFactoryImplementor ); - } - } + /** + * When references are changed, either null or another one, we expect dirty checking to be triggered (both sides + * have to be updated) + * When the internal object is changed, we apply the {Add|Update}Work on containedIns + */ + if (searchForContainers) + { + processContainedIn(entity, queue, rootPropertiesMetadata, searchFactoryImplementor); + } + } - private void processContainedIn(Object instance, List<LuceneWork> queue, PropertiesMetadata metadata, SearchFactoryImplementor searchFactoryImplementor) { - for (int i = 0; i < metadata.containedInGetters.size(); i++) { - XMember member = metadata.containedInGetters.get( i ); - Object value = getMemberValue( instance, member ); - if ( value == null ) continue; + private void processContainedIn(Object instance, List<LuceneWork> queue, PropertiesMetadata metadata, SearchFactoryImplementor searchFactoryImplementor) + { + for (int i = 0; i < metadata.containedInGetters.size(); i++) + { + XMember member = metadata.containedInGetters.get(i); + Object value = getMemberValue(instance, member); + if (value == null) continue; - if ( member.isArray() ) { - for (Object arrayValue : (Object[]) value) { - //highly inneficient but safe wrt the actual targeted class - Class valueClass = Hibernate.getClass( arrayValue ); - DocumentBuilder builder = searchFactoryImplementor.getDocumentBuilders().get( valueClass ); - if ( builder == null ) continue; - processContainedInValue( arrayValue, queue, valueClass, builder, searchFactoryImplementor ); - } - } - else if ( member.isCollection() ) { - Collection collection; - if ( Map.class.equals( member.getCollectionClass() ) ) { - //hum - collection = ( (Map) value ).values(); - } - else { - collection = (Collection) value; - } - for (Object collectionValue : collection) { - //highly inneficient but safe wrt the actual targeted class - Class valueClass = Hibernate.getClass( collectionValue ); - DocumentBuilder builder = searchFactoryImplementor.getDocumentBuilders().get( valueClass ); - if ( builder == null ) continue; - processContainedInValue( collectionValue, queue, valueClass, builder, searchFactoryImplementor ); - } - } - else { - Class valueClass = Hibernate.getClass( value ); - DocumentBuilder builder = searchFactoryImplementor.getDocumentBuilders().get( valueClass ); - if ( builder == null ) continue; - processContainedInValue( value, queue, valueClass, builder, searchFactoryImplementor ); - } - } - //an embedded cannot have a useful @ContainedIn (no shared reference) - //do not walk through them - } + if (member.isArray()) + { + for (Object arrayValue : (Object[]) value) + { + //highly inneficient but safe wrt the actual targeted class + Class valueClass = Hibernate.getClass(arrayValue); + DocumentBuilder builder = searchFactoryImplementor.getDocumentBuilders().get(valueClass); + if (builder == null) continue; + processContainedInValue(arrayValue, queue, valueClass, builder, searchFactoryImplementor); + } + } + else if (member.isCollection()) + { + Collection collection; + if (Map.class.equals(member.getCollectionClass())) + { + //hum + collection = ((Map) value).values(); + } + else + { + collection = (Collection) value; + } + for (Object collectionValue : collection) + { + //highly inneficient but safe wrt the actual targeted class + Class valueClass = Hibernate.getClass(collectionValue); + DocumentBuilder builder = searchFactoryImplementor.getDocumentBuilders().get(valueClass); + if (builder == null) continue; + processContainedInValue(collectionValue, queue, valueClass, builder, searchFactoryImplementor); + } + } + else + { + Class valueClass = Hibernate.getClass(value); + DocumentBuilder builder = searchFactoryImplementor.getDocumentBuilders().get(valueClass); + if (builder == null) continue; + processContainedInValue(value, queue, valueClass, builder, searchFactoryImplementor); + } + } + //an embedded cannot have a useful @ContainedIn (no shared reference) + //do not walk through them + } - private void processContainedInValue(Object value, List<LuceneWork> queue, Class valueClass, - DocumentBuilder builder, SearchFactoryImplementor searchFactoryImplementor) { - Serializable id = (Serializable) builder.getMemberValue( value, builder.idGetter ); - builder.addWorkToQueue( valueClass, value, id, WorkType.UPDATE, queue, searchFactoryImplementor ); - } + private void processContainedInValue(Object value, List<LuceneWork> queue, Class valueClass, + DocumentBuilder builder, SearchFactoryImplementor searchFactoryImplementor) + { + Serializable id = (Serializable) builder.getMemberValue(value, builder.idGetter); + builder.addWorkToQueue(valueClass, value, id, WorkType.UPDATE, queue, searchFactoryImplementor); + } - public Document getDocument(T instance, Serializable id) { - Document doc = new Document(); - XClass instanceClass = reflectionManager.toXClass( Hibernate.getClass( instance ) ); - if ( rootPropertiesMetadata.boost != null ) { - doc.setBoost( rootPropertiesMetadata.boost ); - } - { - Field classField = - new Field( CLASS_FIELDNAME, instanceClass.getName(), Field.Store.YES, Field.Index.UN_TOKENIZED, Field.TermVector.NO ); - doc.add( classField ); - idBridge.set( idKeywordName, id, doc, Field.Store.YES, Field.Index.UN_TOKENIZED, Field.TermVector.NO, idBoost ); - } - buildDocumentFields( instance, doc, rootPropertiesMetadata ); - return doc; - } + public Document getDocument(T instance, Serializable id) + { + Document doc = new Document(); + XClass instanceClass = reflectionManager.toXClass(Hibernate.getClass(instance)); + if (rootPropertiesMetadata.boost != null) + { + doc.setBoost(rootPropertiesMetadata.boost); + } + { + Field classField = + new Field(CLASS_FIELDNAME, instanceClass.getName(), Field.Store.YES, Field.Index.UN_TOKENIZED, Field.TermVector.NO); + doc.add(classField); + idBridge.set(idKeywordName, id, doc, Field.Store.YES, Field.Index.UN_TOKENIZED, Field.TermVector.NO, idBoost); + } + buildDocumentFields(instance, doc, rootPropertiesMetadata); + return doc; + } - private void buildDocumentFields(Object instance, Document doc, PropertiesMetadata propertiesMetadata) { - if ( instance == null ) return; - //needed for field access: I cannot work in the proxied version - Object unproxiedInstance = unproxy( instance ); - for (int i = 0; i < propertiesMetadata.classBridges.size(); i++) { - FieldBridge fb = propertiesMetadata.classBridges.get( i ); + private void buildDocumentFields(Object instance, Document doc, PropertiesMetadata propertiesMetadata) + { + if (instance == null) return; + //needed for field access: I cannot work in the proxied version + Object unproxiedInstance = unproxy(instance); + for (int i = 0; i < propertiesMetadata.classBridges.size(); i++) + { + FieldBridge fb = propertiesMetadata.classBridges.get(i); - fb.set( propertiesMetadata.classNames.get( i ), - unproxiedInstance, - doc, - propertiesMetadata.classStores.get( i ), - propertiesMetadata.classIndexes.get( i ), - propertiesMetadata.classTermVectors.get( i ), - propertiesMetadata.classBoosts.get( i ) ); - } - for (int i = 0; i < propertiesMetadata.fieldNames.size(); i++) { - XMember member = propertiesMetadata.fieldGetters.get( i ); - Object value = getMemberValue( unproxiedInstance, member ); - propertiesMetadata.fieldBridges.get( i ).set( - propertiesMetadata.fieldNames.get( i ), - value, doc, - propertiesMetadata.fieldStore.get( i ), - propertiesMetadata.fieldIndex.get( i ), - propertiesMetadata.fieldTermVectors.get( i ), - getBoost( member ) - ); - } - for (int i = 0; i < propertiesMetadata.embeddedGetters.size(); i++) { - XMember member = propertiesMetadata.embeddedGetters.get( i ); - Object value = getMemberValue( unproxiedInstance, member ); - //TODO handle boost at embedded level: already stored in propertiesMedatada.boost + fb.set(propertiesMetadata.classNames.get(i), + unproxiedInstance, + doc, + propertiesMetadata.classStores.get(i), + propertiesMetadata.classIndexes.get(i), + propertiesMetadata.classTermVectors.get(i), + propertiesMetadata.classBoosts.get(i)); + } + for (int i = 0; i < propertiesMetadata.fieldNames.size(); i++) + { + XMember member = propertiesMetadata.fieldGetters.get(i); + Object value = getMemberValue(unproxiedInstance, member); + propertiesMetadata.fieldBridges.get(i).set( + propertiesMetadata.fieldNames.get(i), + value, doc, + propertiesMetadata.fieldStore.get(i), + propertiesMetadata.fieldIndex.get(i), + propertiesMetadata.fieldTermVectors.get(i), + getBoost(member) + ); + } + for (int i = 0; i < propertiesMetadata.embeddedGetters.size(); i++) + { + XMember member = propertiesMetadata.embeddedGetters.get(i); + Object value = getMemberValue(unproxiedInstance, member); + //TODO handle boost at embedded level: already stored in propertiesMedatada.boost - if ( value == null ) continue; - PropertiesMetadata embeddedMetadata = propertiesMetadata.embeddedPropertiesMetadata.get( i ); - switch ( propertiesMetadata.embeddedContainers.get( i ) ) { - case ARRAY: - for (Object arrayValue : (Object[]) value) { - buildDocumentFields( arrayValue, doc, embeddedMetadata ); - } - break; - case COLLECTION: - for (Object collectionValue : (Collection) value) { - buildDocumentFields( collectionValue, doc, embeddedMetadata ); - } - break; - case MAP: - for (Object collectionValue : ( (Map) value ).values()) { - buildDocumentFields( collectionValue, doc, embeddedMetadata ); - } - break; - case OBJECT: - buildDocumentFields( value, doc, embeddedMetadata ); - break; - default: - throw new AssertionFailure( "Unknown embedded container: " - + propertiesMetadata.embeddedContainers.get( i ) ); - } - } - } + if (value == null) continue; + PropertiesMetadata embeddedMetadata = propertiesMetadata.embeddedPropertiesMetadata.get(i); + switch (propertiesMetadata.embeddedContainers.get(i)) + { + case ARRAY: + for (Object arrayValue : (Object[]) value) + { + buildDocumentFields(arrayValue, doc, embeddedMetadata); + } + break; + case COLLECTION: + for (Object collectionValue : (Collection) value) + { + buildDocumentFields(collectionValue, doc, embeddedMetadata); + } + break; + case MAP: + for (Object collectionValue : ((Map) value).values()) + { + buildDocumentFields(collectionValue, doc, embeddedMetadata); + } + break; + case OBJECT: + buildDocumentFields(value, doc, embeddedMetadata); + break; + default: + throw new AssertionFailure("Unknown embedded container: " + + propertiesMetadata.embeddedContainers.get(i)); + } + } + } - private Object unproxy(Object value) { - //FIXME this service should be part of Core? - if ( value instanceof HibernateProxy ) { - // .getImplementation() initializes the data by side effect - value = ( (HibernateProxy) value ).getHibernateLazyInitializer() - .getImplementation(); - } - return value; - } + private Object unproxy(Object value) + { + //FIXME this service should be part of Core? + if (value instanceof HibernateProxy) + { + // .getImplementation() initializes the data by side effect + value = ((HibernateProxy) value).getHibernateLazyInitializer() + .getImplementation(); + } + return value; + } - public Term getTerm(Serializable id) { - return new Term( idKeywordName, idBridge.objectToString( id ) ); - } + public Term getTerm(Serializable id) + { + return new Term(idKeywordName, idBridge.objectToString(id)); + } - public DirectoryProvider[] getDirectoryProviders() { - return directoryProviders; - } + public DirectoryProvider[] getDirectoryProviders() + { + return directoryProviders; + } - public IndexShardingStrategy getDirectoryProviderSelectionStrategy() { - return shardingStrategy; - } + public IndexShardingStrategy getDirectoryProviderSelectionStrategy() + { + return shardingStrategy; + } - public Analyzer getAnalyzer() { - return analyzer; - } + public Analyzer getAnalyzer() + { + return analyzer; + } - private static void setAccessible(XMember member) { - if ( !Modifier.isPublic( member.getModifiers() ) ) { - member.setAccessible( true ); - } - } + private static void setAccessible(XMember member) + { + if (!Modifier.isPublic(member.getModifiers())) + { + member.setAccessible(true); + } + } - public TwoWayFieldBridge getIdBridge() { - return idBridge; - } + public TwoWayFieldBridge getIdBridge() + { + return idBridge; + } - public String getIdKeywordName() { - return idKeywordName; - } + public String getIdKeywordName() + { + return idKeywordName; + } - public static Class getDocumentClass(Document document) { - String className = document.get( DocumentBuilder.CLASS_FIELDNAME ); - try { - return ReflectHelper.classForName( className ); - } - catch (ClassNotFoundException e) { - throw new SearchException( "Unable to load indexed class: " + className, e ); - } - } + public static Class getDocumentClass(Document document) + { + String className = document.get(DocumentBuilder.CLASS_FIELDNAME); + try + { + return ReflectHelper.classForName(className); + } + catch (ClassNotFoundException e) + { + throw new SearchException("Unable to load indexed class: " + className, e); + } + } - public static Serializable getDocumentId(SearchFactoryImplementor searchFactoryImplementor, Class clazz, Document document) { - DocumentBuilder builder = searchFactoryImplementor.getDocumentBuilders().get( clazz ); - if ( builder == null ) throw new SearchException( "No Lucene configuration set up for: " + clazz.getName() ); - return (Serializable) builder.getIdBridge().get( builder.getIdKeywordName(), document ); - } + public static Serializable getDocumentId(SearchFactoryImplementor searchFactoryImplementor, Class clazz, Document document) + { + DocumentBuilder builder = searchFactoryImplementor.getDocumentBuilders().get(clazz); + if (builder == null) throw new SearchException("No Lucene configuration set up for: " + clazz.getName()); + return (Serializable) builder.getIdBridge().get(builder.getIdKeywordName(), document); + } - public static Object[] getDocumentFields(SearchFactoryImplementor searchFactoryImplementor, Class clazz, Document document, String[] fields) { - DocumentBuilder builder = searchFactoryImplementor.getDocumentBuilders().get( clazz ); - if ( builder == null ) throw new SearchException( "No Lucene configuration set up for: " + clazz.getName() ); - final int fieldNbr = fields.length; - Object[] result = new Object[fieldNbr]; + public static Object[] getDocumentFields(SearchFactoryImplementor searchFactoryImplementor, Class clazz, Document document, String[] fields) + { + DocumentBuilder builder = searchFactoryImplementor.getDocumentBuilders().get(clazz); + if (builder == null) throw new SearchException("No Lucene configuration set up for: " + clazz.getName()); + final int fieldNbr = fields.length; + Object[] result = new Object[fieldNbr]; - if ( builder.idKeywordName != null ) { - populateResult( builder.idKeywordName, builder.idBridge, Field.Store.YES, fields, result, document ); - } + if (builder.idKeywordName != null) + { + populateResult(builder.idKeywordName, builder.idBridge, Field.Store.YES, fields, result, document); + } - final PropertiesMetadata metadata = builder.rootPropertiesMetadata; - processFieldsForProjection( metadata, fields, result, document ); - return result; - } + final PropertiesMetadata metadata = builder.rootPropertiesMetadata; + processFieldsForProjection(metadata, fields, result, document); + return result; + } - private static void processFieldsForProjection(PropertiesMetadata metadata, String[] fields, Object[] result, Document document) { - final int nbrFoEntityFields = metadata.fieldNames.size(); - for (int index = 0; index < nbrFoEntityFields; index++) { - populateResult( metadata.fieldNames.get( index ), - metadata.fieldBridges.get( index ), - metadata.fieldStore.get( index ), - fields, - result, - document - ); - } - final int nbrOfEmbeddedObjects = metadata.embeddedPropertiesMetadata.size(); - for (int index = 0; index < nbrOfEmbeddedObjects; index++) { - //there is nothing we can do for collections - if ( metadata.embeddedContainers.get( index ) == PropertiesMetadata.Container.OBJECT ) { - processFieldsForProjection( metadata.embeddedPropertiesMetadata.get( index ), fields, result, document ); - } - } - } + private static void processFieldsForProjection(PropertiesMetadata metadata, String[] fields, Object[] result, Document document) + { + final int nbrFoEntityFields = metadata.fieldNames.size(); + for (int index = 0; index < nbrFoEntityFields; index++) + { + populateResult(metadata.fieldNames.get(index), + metadata.fieldBridges.get(index), + metadata.fieldStore.get(index), + fields, + result, + document + ); + } + final int nbrOfEmbeddedObjects = metadata.embeddedPropertiesMetadata.size(); + for (int index = 0; index < nbrOfEmbeddedObjects; index++) + { + //there is nothing we can do for collections + if (metadata.embeddedContainers.get(index) == PropertiesMetadata.Container.OBJECT) + { + processFieldsForProjection(metadata.embeddedPropertiesMetadata.get(index), fields, result, document); + } + } + } - private static void populateResult(String fieldName, FieldBridge fieldBridge, Field.Store store, - String[] fields, Object[] result, Document document) { - int matchingPosition = getFieldPosition( fields, fieldName ); - if ( matchingPosition != -1 ) { - //TODO make use of an isTwoWay() method - if ( store != Field.Store.NO && TwoWayFieldBridge.class.isAssignableFrom( fieldBridge.getClass() ) ) { - result[matchingPosition] = ( (TwoWayFieldBridge) fieldBridge ).get( fieldName, document ); - if ( log.isTraceEnabled() ) { - log.trace( "Field {} projected as {}", fieldName, result[matchingPosition] ); - } - } - else { - if ( store == Field.Store.NO ) { - throw new SearchException( "Projecting an unstored field: " + fieldName ); - } - else { - throw new SearchException( "FieldBridge is not a TwoWayFieldBridge: " + fieldBridge.getClass() ); - } - } - } - } + private static void populateResult(String fieldName, FieldBridge fieldBridge, Field.Store store, + String[] fields, Object[] result, Document document) + { + int matchingPosition = getFieldPosition(fields, fieldName); + if (matchingPosition != -1) + { + //TODO make use of an isTwoWay() method + if (store != Field.Store.NO && TwoWayFieldBridge.class.isAssignableFrom(fieldBridge.getClass())) + { + result[matchingPosition] = ((TwoWayFieldBridge) fieldBridge).get(fieldName, document); + if (log.isTraceEnabled()) + { + log.trace("Field {} projected as {}", fieldName, result[matchingPosition]); + } + } + else + { + if (store == Field.Store.NO) + { + throw new SearchException("Projecting an unstored field: " + fieldName); + } + else + { + throw new SearchException("FieldBridge is not a TwoWayFieldBridge: " + fieldBridge.getClass()); + } + } + } + } - private static int getFieldPosition(String[] fields, String fieldName) { - int fieldNbr = fields.length; - for (int index = 0; index < fieldNbr; index++) { - if ( fieldName.equals( fields[index] ) ) return index; - } - return -1; - } + private static int getFieldPosition(String[] fields, String fieldName) + { + int fieldNbr = fields.length; + for (int index = 0; index < fieldNbr; index++) + { + if (fieldName.equals(fields[index])) return index; + } + return -1; + } - public void postInitialize(Set<Class> indexedClasses) { - //this method does not requires synchronization - Class plainClass = reflectionManager.toClass( beanClass ); - Set<Class> tempMappedSubclasses = new HashSet<Class>(); - //together with the caller this creates a o(2), but I think it's still faster than create the up hierarchy for each class - for (Class currentClass : indexedClasses) { - if ( plainClass.isAssignableFrom( currentClass ) ) tempMappedSubclasses.add( currentClass ); - } - this.mappedSubclasses = Collections.unmodifiableSet( tempMappedSubclasses ); - Class superClass = plainClass.getSuperclass(); - this.isRoot = true; - while ( superClass != null) { - if ( indexedClasses.contains( superClass ) ) { - this.isRoot = false; - break; - } - superClass = superClass.getSuperclass(); - } - } + public void postInitialize(Set<Class> indexedClasses) + { + //this method does not requires synchronization + Class plainClass = reflectionManager.toClass(beanClass); + Set<Class> tempMappedSubclasses = new HashSet<Class>(); + //together with the caller this creates a o(2), but I think it's still faster than create the up hierarchy for each class + for (Class currentClass : indexedClasses) + { + if (plainClass.isAssignableFrom(currentClass)) tempMappedSubclasses.add(currentClass); + } + this.mappedSubclasses = Collections.unmodifiableSet(tempMappedSubclasses); + Class superClass = plainClass.getSuperclass(); + this.isRoot = true; + while (superClass != null) + { + if (indexedClasses.contains(superClass)) + { + this.isRoot = false; + break; + } + superClass = superClass.getSuperclass(); + } + } - public Set<Class> getMappedSubclasses() { - return mappedSubclasses; - } + public Set<Class> getMappedSubclasses() + { + return mappedSubclasses; + } - /** - * Make sure to return false if there is a risk of composite id - * if composite id, use of (a, b) in ((1,2), (3,4)) fails on most database - */ - public boolean isSafeFromTupleId() { - return safeFromTupleId; - } + /** + * Make sure to return false if there is a risk of composite id + * if composite id, use of (a, b) in ((1,2), (3,4)) fails on most database + */ + public boolean isSafeFromTupleId() + { + return safeFromTupleId; + } - private static class PropertiesMetadata { - public Float boost; - public Analyzer analyzer; - public final List<String> fieldNames = new ArrayList<String>(); - public final List<XMember> fieldGetters = new ArrayList<XMember>(); - public final List<FieldBridge> fieldBridges = new ArrayList<FieldBridge>(); - public final List<Field.Store> fieldStore = new ArrayList<Field.Store>(); - public final List<Field.Index> fieldIndex = new ArrayList<Field.Index>(); - public final List<Field.TermVector> fieldTermVectors = new ArrayList<Field.TermVector>(); - public final List<XMember> embeddedGetters = new ArrayList<XMember>(); - public final List<PropertiesMetadata> embeddedPropertiesMetadata = new ArrayList<PropertiesMetadata>(); - public final List<Container> embeddedContainers = new ArrayList<Container>(); - public final List<XMember> containedInGetters = new ArrayList<XMember>(); - public final List<String> classNames = new ArrayList<String>(); - public final List<Field.Store> classStores = new ArrayList<Field.Store>(); - public final List<Field.Index> classIndexes = new ArrayList<Field.Index>(); - public final List<FieldBridge> classBridges = new ArrayList<FieldBridge>(); - public final List<Field.TermVector> classTermVectors = new ArrayList<Field.TermVector>(); - public final List<Float> classBoosts = new ArrayList<Float>(); + private static class PropertiesMetadata + { + public Float boost; + public Analyzer analyzer; + public final List<String> fieldNames = new ArrayList<String>(); + public final List<XMember> fieldGetters = new ArrayList<XMember>(); + public final List<FieldBridge> fieldBridges = new ArrayList<FieldBridge>(); + public final List<Field.Store> fieldStore = new ArrayList<Field.Store>(); + public final List<Field.Index> fieldIndex = new ArrayList<Field.Index>(); + public final List<Field.TermVector> fieldTermVectors = new ArrayList<Field.TermVector>(); + public final List<XMember> embeddedGetters = new ArrayList<XMember>(); + public final List<PropertiesMetadata> embeddedPropertiesMetadata = new ArrayList<PropertiesMetadata>(); + public final List<Container> embeddedContainers = new ArrayList<Container>(); + public final List<XMember> containedInGetters = new ArrayList<XMember>(); + public final List<String> classNames = new ArrayList<String>(); + public final List<Field.Store> classStores = new ArrayList<Field.Store>(); + public final List<Field.Index> classIndexes = new ArrayList<Field.Index>(); + public final List<FieldBridge> classBridges = new ArrayList<FieldBridge>(); + public final List<Field.TermVector> classTermVectors = new ArrayList<Field.TermVector>(); + public final List<Float> classBoosts = new ArrayList<Float>(); - public enum Container { - OBJECT, - COLLECTION, - MAP, - ARRAY - } - } + public enum Container + { + OBJECT, + COLLECTION, + MAP, + ARRAY + } + } }