diff --git a/src/main/asciidoc/reference/repositories.adoc b/src/main/asciidoc/reference/repositories.adoc index 0daf77b0..8e40b0a5 100644 --- a/src/main/asciidoc/reference/repositories.adoc +++ b/src/main/asciidoc/reference/repositories.adoc @@ -196,7 +196,7 @@ As an example, suppose you have a `Customers` application domain type and corres package ...; import org.springframework.data.annotation.Id; -import org.springframework.data.gemfire.mapping.Region; +import org.springframework.data.gemfire.mapping.annotation.Region; ... @Region("Customers") diff --git a/src/main/java/org/springframework/data/gemfire/config/annotation/EnableEntityDefinedRegions.java b/src/main/java/org/springframework/data/gemfire/config/annotation/EnableEntityDefinedRegions.java index 84551019..9a798336 100644 --- a/src/main/java/org/springframework/data/gemfire/config/annotation/EnableEntityDefinedRegions.java +++ b/src/main/java/org/springframework/data/gemfire/config/annotation/EnableEntityDefinedRegions.java @@ -63,7 +63,7 @@ public @interface EnableEntityDefinedRegions { String[] value() default {}; /** - * Base packages to scan for {@link org.springframework.data.gemfire.mapping.Region @Region} annotated + * Base packages to scan for {@link org.springframework.data.gemfire.mapping.annotation.Region @Region} annotated * application persistent entities. {@link #value()} is an alias for this attribute. * Use {@link #basePackageClasses()} for a type-safe alternative to String-based package names. * @@ -75,7 +75,7 @@ public @interface EnableEntityDefinedRegions { /** * Type-safe alternative to {@link #basePackages()} for specifying the packages to scan for - * {@link org.springframework.data.gemfire.mapping.Region @Region} annotated application persistent entities. + * {@link org.springframework.data.gemfire.mapping.annotation.Region @Region} annotated application persistent entities. * The package of each class specified will be scanned. Consider creating a special no-op marker class or interface * in each package that serves no other purpose than being referenced by this attribute. * diff --git a/src/main/java/org/springframework/data/gemfire/config/annotation/EntityDefinedRegionsConfiguration.java b/src/main/java/org/springframework/data/gemfire/config/annotation/EntityDefinedRegionsConfiguration.java index 2da55f07..de4d74a2 100644 --- a/src/main/java/org/springframework/data/gemfire/config/annotation/EntityDefinedRegionsConfiguration.java +++ b/src/main/java/org/springframework/data/gemfire/config/annotation/EntityDefinedRegionsConfiguration.java @@ -61,12 +61,12 @@ import org.springframework.data.gemfire.client.ClientRegionFactoryBean; import org.springframework.data.gemfire.config.annotation.support.GemFireCacheTypeAwareRegionFactoryBean; import org.springframework.data.gemfire.config.annotation.support.GemFireComponentClassTypeScanner; import org.springframework.data.gemfire.config.xml.GemfireConstants; -import org.springframework.data.gemfire.mapping.ClientRegion; import org.springframework.data.gemfire.mapping.GemfireMappingContext; import org.springframework.data.gemfire.mapping.GemfirePersistentEntity; -import org.springframework.data.gemfire.mapping.LocalRegion; -import org.springframework.data.gemfire.mapping.PartitionRegion; -import org.springframework.data.gemfire.mapping.ReplicateRegion; +import org.springframework.data.gemfire.mapping.annotation.ClientRegion; +import org.springframework.data.gemfire.mapping.annotation.LocalRegion; +import org.springframework.data.gemfire.mapping.annotation.PartitionRegion; +import org.springframework.data.gemfire.mapping.annotation.ReplicateRegion; import org.springframework.data.gemfire.util.ArrayUtils; import org.springframework.data.gemfire.util.CollectionUtils; import org.springframework.util.Assert; @@ -93,13 +93,13 @@ import org.springframework.util.StringUtils; * @see org.springframework.data.gemfire.client.ClientRegionFactoryBean * @see org.springframework.data.gemfire.config.annotation.support.GemFireCacheTypeAwareRegionFactoryBean * @see org.springframework.data.gemfire.config.annotation.support.GemFireComponentClassTypeScanner - * @see org.springframework.data.gemfire.mapping.ClientRegion + * @see ClientRegion * @see org.springframework.data.gemfire.mapping.GemfireMappingContext * @see org.springframework.data.gemfire.mapping.GemfirePersistentEntity - * @see org.springframework.data.gemfire.mapping.LocalRegion - * @see org.springframework.data.gemfire.mapping.PartitionRegion - * @see org.springframework.data.gemfire.mapping.ReplicateRegion - * @see org.springframework.data.gemfire.mapping.Region + * @see LocalRegion + * @see PartitionRegion + * @see ReplicateRegion + * @see org.springframework.data.gemfire.mapping.annotation.Region * @since 1.9.0 */ public class EntityDefinedRegionsConfiguration @@ -116,7 +116,7 @@ public class EntityDefinedRegionsConfiguration regionAnnotationToRegionFactoryBeanClass.put(LocalRegion.class, LocalRegionFactoryBean.class); regionAnnotationToRegionFactoryBeanClass.put(PartitionRegion.class, PartitionedRegionFactoryBean.class); regionAnnotationToRegionFactoryBeanClass.put(ReplicateRegion.class, ReplicatedRegionFactoryBean.class); - regionAnnotationToRegionFactoryBeanClass.put(org.springframework.data.gemfire.mapping.Region.class, + regionAnnotationToRegionFactoryBeanClass.put(org.springframework.data.gemfire.mapping.annotation.Region.class, DEFAULT_REGION_FACTORY_BEAN_CLASS); } @@ -386,7 +386,7 @@ public class EntityDefinedRegionsConfiguration Set regionAnnotatedPersistentEntityTypeFilters = new HashSet<>(); for (Class annotationType : - org.springframework.data.gemfire.mapping.Region.REGION_ANNOTATION_TYPES) { + org.springframework.data.gemfire.mapping.annotation.Region.REGION_ANNOTATION_TYPES) { regionAnnotatedPersistentEntityTypeFilters.add(new AnnotationTypeFilter(annotationType)); } diff --git a/src/main/java/org/springframework/data/gemfire/config/annotation/IndexConfiguration.java b/src/main/java/org/springframework/data/gemfire/config/annotation/IndexConfiguration.java index 4233e820..9e2d6faa 100644 --- a/src/main/java/org/springframework/data/gemfire/config/annotation/IndexConfiguration.java +++ b/src/main/java/org/springframework/data/gemfire/config/annotation/IndexConfiguration.java @@ -31,7 +31,7 @@ import org.springframework.data.gemfire.IndexType; import org.springframework.data.gemfire.config.xml.GemfireConstants; import org.springframework.data.gemfire.mapping.GemfirePersistentEntity; import org.springframework.data.gemfire.mapping.GemfirePersistentProperty; -import org.springframework.data.gemfire.mapping.Indexed; +import org.springframework.data.gemfire.mapping.annotation.Indexed; import org.springframework.data.mapping.PropertyHandler; import org.springframework.util.StringUtils; @@ -51,7 +51,7 @@ import org.springframework.util.StringUtils; * @see org.springframework.data.gemfire.config.annotation.EntityDefinedRegionsConfiguration * @see org.springframework.data.gemfire.mapping.GemfirePersistentEntity * @see org.springframework.data.gemfire.mapping.GemfirePersistentProperty - * @see org.springframework.data.gemfire.mapping.Indexed + * @see Indexed * @see org.apache.geode.cache.Region * @see org.apache.geode.cache.query.Index * @since 1.9.0 diff --git a/src/main/java/org/springframework/data/gemfire/mapping/GemfirePersistentEntity.java b/src/main/java/org/springframework/data/gemfire/mapping/GemfirePersistentEntity.java index c7e3151a..ac881f4e 100644 --- a/src/main/java/org/springframework/data/gemfire/mapping/GemfirePersistentEntity.java +++ b/src/main/java/org/springframework/data/gemfire/mapping/GemfirePersistentEntity.java @@ -23,6 +23,11 @@ import java.lang.annotation.Annotation; import org.springframework.core.annotation.AnnotatedElementUtils; import org.springframework.core.annotation.AnnotationAttributes; import org.springframework.core.annotation.AnnotationUtils; +import org.springframework.data.gemfire.mapping.annotation.ClientRegion; +import org.springframework.data.gemfire.mapping.annotation.LocalRegion; +import org.springframework.data.gemfire.mapping.annotation.PartitionRegion; +import org.springframework.data.gemfire.mapping.annotation.Region; +import org.springframework.data.gemfire.mapping.annotation.ReplicateRegion; import org.springframework.data.mapping.PersistentEntity; import org.springframework.data.mapping.model.BasicPersistentEntity; import org.springframework.data.util.TypeInformation; @@ -88,11 +93,11 @@ public class GemfirePersistentEntity extends BasicPersistentEntity concrete {@link Class} type of the Region {@link Annotation}. * @return the {@link Region} annotation used to annotate this {@link PersistentEntity} or {@literal null} * if this {@link PersistentEntity} was not annotated with a {@link Region} annotation. - * @see org.springframework.data.gemfire.mapping.ClientRegion - * @see org.springframework.data.gemfire.mapping.LocalRegion - * @see org.springframework.data.gemfire.mapping.PartitionRegion - * @see org.springframework.data.gemfire.mapping.ReplicateRegion - * @see org.springframework.data.gemfire.mapping.Region + * @see ClientRegion + * @see LocalRegion + * @see PartitionRegion + * @see ReplicateRegion + * @see Region * @see java.lang.annotation.Annotation */ @SuppressWarnings("unchecked") diff --git a/src/main/java/org/springframework/data/gemfire/mapping/ClientRegion.java b/src/main/java/org/springframework/data/gemfire/mapping/annotation/ClientRegion.java similarity index 97% rename from src/main/java/org/springframework/data/gemfire/mapping/ClientRegion.java rename to src/main/java/org/springframework/data/gemfire/mapping/annotation/ClientRegion.java index 68a665f4..e5f9075b 100644 --- a/src/main/java/org/springframework/data/gemfire/mapping/ClientRegion.java +++ b/src/main/java/org/springframework/data/gemfire/mapping/annotation/ClientRegion.java @@ -15,7 +15,7 @@ * */ -package org.springframework.data.gemfire.mapping; +package org.springframework.data.gemfire.mapping.annotation; import java.lang.annotation.Annotation; import java.lang.annotation.Documented; @@ -35,7 +35,7 @@ import org.springframework.data.gemfire.config.xml.GemfireConstants; * persistent entity will be stored. * * @author John Blum - * @see org.springframework.data.gemfire.mapping.Region + * @see Region * @since 1.9.0 */ @Target(ElementType.TYPE) diff --git a/src/main/java/org/springframework/data/gemfire/mapping/Indexed.java b/src/main/java/org/springframework/data/gemfire/mapping/annotation/Indexed.java similarity index 91% rename from src/main/java/org/springframework/data/gemfire/mapping/Indexed.java rename to src/main/java/org/springframework/data/gemfire/mapping/annotation/Indexed.java index ba5ae720..043c7b21 100644 --- a/src/main/java/org/springframework/data/gemfire/mapping/Indexed.java +++ b/src/main/java/org/springframework/data/gemfire/mapping/annotation/Indexed.java @@ -15,7 +15,7 @@ * */ -package org.springframework.data.gemfire.mapping; +package org.springframework.data.gemfire.mapping.annotation; import java.lang.annotation.Documented; import java.lang.annotation.ElementType; @@ -27,6 +27,8 @@ import java.lang.annotation.Target; import org.apache.geode.cache.query.Index; import org.springframework.core.annotation.AliasFor; import org.springframework.data.gemfire.IndexType; +import org.springframework.data.gemfire.mapping.GemfirePersistentEntity; +import org.springframework.data.gemfire.mapping.GemfirePersistentProperty; /** * The {@link Indexed} annotation is used to index a {@link GemfirePersistentEntity} {@link GemfirePersistentProperty}, diff --git a/src/main/java/org/springframework/data/gemfire/mapping/LocalRegion.java b/src/main/java/org/springframework/data/gemfire/mapping/annotation/LocalRegion.java similarity index 97% rename from src/main/java/org/springframework/data/gemfire/mapping/LocalRegion.java rename to src/main/java/org/springframework/data/gemfire/mapping/annotation/LocalRegion.java index eb0c0bc5..512f31b0 100644 --- a/src/main/java/org/springframework/data/gemfire/mapping/LocalRegion.java +++ b/src/main/java/org/springframework/data/gemfire/mapping/annotation/LocalRegion.java @@ -15,7 +15,7 @@ * */ -package org.springframework.data.gemfire.mapping; +package org.springframework.data.gemfire.mapping.annotation; import java.lang.annotation.Annotation; import java.lang.annotation.Documented; @@ -31,7 +31,7 @@ import org.springframework.core.annotation.AliasFor; * {@link Annotation} defining the Local {@link Region} in which the application persistent entity will be stored. * * @author John Blum - * @see org.springframework.data.gemfire.mapping.Region + * @see Region * @since 1.9.0 */ @Target(ElementType.TYPE) diff --git a/src/main/java/org/springframework/data/gemfire/mapping/PartitionRegion.java b/src/main/java/org/springframework/data/gemfire/mapping/annotation/PartitionRegion.java similarity index 98% rename from src/main/java/org/springframework/data/gemfire/mapping/PartitionRegion.java rename to src/main/java/org/springframework/data/gemfire/mapping/annotation/PartitionRegion.java index 80945356..744b0401 100644 --- a/src/main/java/org/springframework/data/gemfire/mapping/PartitionRegion.java +++ b/src/main/java/org/springframework/data/gemfire/mapping/annotation/PartitionRegion.java @@ -15,7 +15,7 @@ * */ -package org.springframework.data.gemfire.mapping; +package org.springframework.data.gemfire.mapping.annotation; import java.lang.annotation.Annotation; import java.lang.annotation.Documented; @@ -31,7 +31,7 @@ import org.springframework.core.annotation.AliasFor; * {@link Annotation} defining the Partition {@link Region} in which the application persistent entity will be stored. * * @author John Blum - * @see org.springframework.data.gemfire.mapping.Region + * @see Region * @since 1.9.0 */ @Target(ElementType.TYPE) diff --git a/src/main/java/org/springframework/data/gemfire/mapping/Region.java b/src/main/java/org/springframework/data/gemfire/mapping/annotation/Region.java similarity index 95% rename from src/main/java/org/springframework/data/gemfire/mapping/Region.java rename to src/main/java/org/springframework/data/gemfire/mapping/annotation/Region.java index 89e6c4c6..e682214e 100644 --- a/src/main/java/org/springframework/data/gemfire/mapping/Region.java +++ b/src/main/java/org/springframework/data/gemfire/mapping/annotation/Region.java @@ -1,5 +1,5 @@ /* - * Copyright 2012 the original author or authors. + * Copyright 2016 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -12,9 +12,10 @@ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. * See the License for the specific language governing permissions and * limitations under the License. + * */ -package org.springframework.data.gemfire.mapping; +package org.springframework.data.gemfire.mapping.annotation; import java.lang.annotation.Annotation; import java.lang.annotation.Documented; diff --git a/src/main/java/org/springframework/data/gemfire/mapping/ReplicateRegion.java b/src/main/java/org/springframework/data/gemfire/mapping/annotation/ReplicateRegion.java similarity index 97% rename from src/main/java/org/springframework/data/gemfire/mapping/ReplicateRegion.java rename to src/main/java/org/springframework/data/gemfire/mapping/annotation/ReplicateRegion.java index 0d71df84..656258a1 100644 --- a/src/main/java/org/springframework/data/gemfire/mapping/ReplicateRegion.java +++ b/src/main/java/org/springframework/data/gemfire/mapping/annotation/ReplicateRegion.java @@ -15,7 +15,7 @@ * */ -package org.springframework.data.gemfire.mapping; +package org.springframework.data.gemfire.mapping.annotation; import java.lang.annotation.Annotation; import java.lang.annotation.Documented; @@ -32,7 +32,7 @@ import org.springframework.data.gemfire.ScopeType; * {@link Annotation} defining the Replicate {@link Region} in which the application persistent entity will be stored. * * @author John Blum - * @see org.springframework.data.gemfire.mapping.Region + * @see Region * @since 1.9.0 */ @Target(ElementType.TYPE) diff --git a/src/main/java/org/springframework/data/gemfire/repository/config/GemfireRepositoryConfigurationExtension.java b/src/main/java/org/springframework/data/gemfire/repository/config/GemfireRepositoryConfigurationExtension.java index ef7f740f..2aee4b3f 100644 --- a/src/main/java/org/springframework/data/gemfire/repository/config/GemfireRepositoryConfigurationExtension.java +++ b/src/main/java/org/springframework/data/gemfire/repository/config/GemfireRepositoryConfigurationExtension.java @@ -24,7 +24,7 @@ import org.springframework.beans.factory.support.BeanDefinitionBuilder; import org.springframework.beans.factory.support.BeanDefinitionRegistry; import org.springframework.beans.factory.support.RootBeanDefinition; import org.springframework.data.gemfire.mapping.GemfireMappingContext; -import org.springframework.data.gemfire.mapping.Region; +import org.springframework.data.gemfire.mapping.annotation.Region; import org.springframework.data.gemfire.repository.GemfireRepository; import org.springframework.data.gemfire.repository.support.GemfireRepositoryFactoryBean; import org.springframework.data.repository.config.AnnotationRepositoryConfigurationSource; diff --git a/src/main/java/org/springframework/data/gemfire/repository/support/GemfireRepositoryFactory.java b/src/main/java/org/springframework/data/gemfire/repository/support/GemfireRepositoryFactory.java index 6e855e3a..39cf340b 100644 --- a/src/main/java/org/springframework/data/gemfire/repository/support/GemfireRepositoryFactory.java +++ b/src/main/java/org/springframework/data/gemfire/repository/support/GemfireRepositoryFactory.java @@ -126,8 +126,8 @@ public class GemfireRepositoryFactory extends RepositoryFactorySupport { } String getRepositoryRegionName(Class repositoryInterface) { - return (repositoryInterface.isAnnotationPresent(org.springframework.data.gemfire.mapping.Region.class) ? - repositoryInterface.getAnnotation(org.springframework.data.gemfire.mapping.Region.class).value() : null); + return (repositoryInterface.isAnnotationPresent(org.springframework.data.gemfire.mapping.annotation.Region.class) ? + repositoryInterface.getAnnotation(org.springframework.data.gemfire.mapping.annotation.Region.class).value() : null); } /* diff --git a/src/main/resources/org/springframework/data/gemfire/config/spring-data-gemfire-2.0.xsd b/src/main/resources/org/springframework/data/gemfire/config/spring-data-gemfire-2.0.xsd index 45383859..8ddd8729 100644 --- a/src/main/resources/org/springframework/data/gemfire/config/spring-data-gemfire-2.0.xsd +++ b/src/main/resources/org/springframework/data/gemfire/config/spring-data-gemfire-2.0.xsd @@ -1,6 +1,5 @@ , SampleCustomRepository { } - @org.springframework.data.gemfire.mapping.Region("People") + @org.springframework.data.gemfire.mapping.annotation.Region("People") interface PersonRepository extends GemfireRepository { } } diff --git a/src/test/java/org/springframework/data/gemfire/test/model/Person.java b/src/test/java/org/springframework/data/gemfire/test/model/Person.java index 0c838a37..48e25d7d 100644 --- a/src/test/java/org/springframework/data/gemfire/test/model/Person.java +++ b/src/test/java/org/springframework/data/gemfire/test/model/Person.java @@ -24,7 +24,7 @@ import java.util.Date; import org.springframework.data.annotation.Id; import org.springframework.data.annotation.PersistenceConstructor; -import org.springframework.data.gemfire.mapping.Region; +import org.springframework.data.gemfire.mapping.annotation.Region; import org.springframework.data.gemfire.test.support.IdentifierSequence; import org.springframework.data.gemfire.util.SpringUtils; import org.springframework.util.Assert;