geode-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From dinesh akhand <dines...@amdocs.com>
Subject Review Request 60678: GEODE-3151 Configuration Parameter Based Registration Of internal Region over JMX
Date Thu, 06 Jul 2017 07:04:35 GMT

-----------------------------------------------------------
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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message