From 6145bdb0775b623c26d51f7e1e2de912808acee4 Mon Sep 17 00:00:00 2001 From: John Blum Date: Wed, 14 Jul 2021 14:43:07 -0700 Subject: [PATCH] Enable statistics for caching-defined Regions. Resolves gh-518. --- .../annotation/CachingDefinedRegionsConfiguration.java | 6 ++++-- .../config/annotation/EnableCachingDefinedRegions.java | 1 - 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/spring-data-geode/src/main/java/org/springframework/data/gemfire/config/annotation/CachingDefinedRegionsConfiguration.java b/spring-data-geode/src/main/java/org/springframework/data/gemfire/config/annotation/CachingDefinedRegionsConfiguration.java index 7cfc731c..37138efa 100644 --- a/spring-data-geode/src/main/java/org/springframework/data/gemfire/config/annotation/CachingDefinedRegionsConfiguration.java +++ b/spring-data-geode/src/main/java/org/springframework/data/gemfire/config/annotation/CachingDefinedRegionsConfiguration.java @@ -117,10 +117,10 @@ public class CachingDefinedRegionsConfiguration extends AbstractAnnotationConfig private ClientRegionShortcut clientRegionShortcut = ClientRegionShortcut.PROXY; - private CompositeLifecycle compositeLifecycle = new CompositeLifecycle(); + private final CompositeLifecycle compositeLifecycle = new CompositeLifecycle(); @Autowired(required = false) - private List regionConfigurers = Collections.emptyList(); + private final List regionConfigurers = Collections.emptyList(); private RegionShortcut serverRegionShortcut = RegionShortcut.PARTITION; @@ -314,6 +314,7 @@ public class CachingDefinedRegionsConfiguration extends AbstractAnnotationConfig regionFactoryBean.setRegionConfigurers(resolveRegionConfigurers()); regionFactoryBean.setRegionName(cacheName); regionFactoryBean.setServerRegionShortcut(resolveServerRegionShortcut()); + regionFactoryBean.setStatisticsEnabled(true); String poolName = resolvePoolName(); @@ -391,6 +392,7 @@ public class CachingDefinedRegionsConfiguration extends AbstractAnnotationConfig protected abstract Class[] getMethodCacheAnnotationTypes(); + @SuppressWarnings("rawtypes") protected Class[] append(Class[] annotationTypes, Class... additionalAnnotationTypes) { List annotationTypeList = new ArrayList<>(Arrays.asList(annotationTypes)); diff --git a/spring-data-geode/src/main/java/org/springframework/data/gemfire/config/annotation/EnableCachingDefinedRegions.java b/spring-data-geode/src/main/java/org/springframework/data/gemfire/config/annotation/EnableCachingDefinedRegions.java index 49054ecb..5dc74ba8 100644 --- a/spring-data-geode/src/main/java/org/springframework/data/gemfire/config/annotation/EnableCachingDefinedRegions.java +++ b/spring-data-geode/src/main/java/org/springframework/data/gemfire/config/annotation/EnableCachingDefinedRegions.java @@ -13,7 +13,6 @@ * See the License for the specific language governing permissions and * limitations under the License. */ - package org.springframework.data.gemfire.config.annotation; import java.lang.annotation.Documented;