-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60678/
-----------------------------------------------------------
Review request for geode and Barry Oglesby.
Bugs: GEODE-3151
https://issues.apache.org/jira/browse/GEODE-3151
Repository: geode
Description
-------
I am Adding the Configuration parameter which will have the internal region
name/function name
which we want to register in JMX.
currently for Internal Region Bean is not getting registered in JMX.
I want to make this as configurable.
If user provided internal region names in configuration[property file ] then
bean should get register in JMX.
Current use: we want to register the async queue associated internal region in
JMX.
future scope: using same parameter we want to register few functions to JMX.
output : only defined internal regions in property file are going to be show in
JMX.
Diffs
-----
geode-core/src/main/java/org/apache/geode/distributed/ConfigurationProperties.java
63f6505101
geode-core/src/main/java/org/apache/geode/distributed/internal/AbstractDistributionConfig.java
795f6a5a4a
geode-core/src/main/java/org/apache/geode/distributed/internal/DistributionConfig.java
c2a395de0b
geode-core/src/main/java/org/apache/geode/distributed/internal/DistributionConfigImpl.java
fbe894c964
geode-core/src/main/java/org/apache/geode/distributed/internal/InternalDistributedSystem.java
6aa602fc8f
geode-core/src/main/java/org/apache/geode/internal/cache/GemFireCacheImpl.java
2dda38c70b
geode-core/src/main/java/org/apache/geode/internal/cache/LocalRegion.java
3b3047ff00
geode-core/src/main/java/org/apache/geode/internal/i18n/LocalizedStrings.java
baad039bf9
geode-core/src/test/java/org/apache/geode/distributed/internal/DistributionConfigJUnitTest.java
525f988cd3
geode-core/src/test/java/org/apache/geode/distributed/internal/InternalDistributedSystemJUnitTest.java
5a191bbd93
Diff: https://reviews.apache.org/r/60678/diff/1/
Testing
-------
Manual testing was done for it.
Added test case for it.
Thanks,
dinesh akhand
