geode-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From n...@apache.org
Subject [geode] branch develop updated: GEODE-5975: Fixed container access. (#2771)
Date Mon, 05 Nov 2018 18:35:17 GMT
This is an automated email from the ASF dual-hosted git repository.

nnag pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/develop by this push:
     new 7405d3d  GEODE-5975: Fixed container access. (#2771)
7405d3d is described below

commit 7405d3d78cbdc771da364d78f4c5f8e7db3125e7
Author: Nabarun Nag <nabarunnag@users.noreply.github.com>
AuthorDate: Mon Nov 5 10:35:01 2018 -0800

    GEODE-5975: Fixed container access. (#2771)
    
    * Ensured that container access is done with the proper object type as a parameter.
---
 .../org/apache/geode/management/internal/ManagementResourceRepo.java    | 2 +-
 .../src/main/java/org/apache/geode/redis/internal/RegionProvider.java   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/ManagementResourceRepo.java
b/geode-core/src/main/java/org/apache/geode/management/internal/ManagementResourceRepo.java
index 4eff14c..137446a 100644
--- a/geode-core/src/main/java/org/apache/geode/management/internal/ManagementResourceRepo.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/ManagementResourceRepo.java
@@ -132,7 +132,7 @@ public class ManagementResourceRepo {
    */
   public Object getEntryFromLocalMonitoringRegion(ObjectName name) {
 
-    return localMonitoringRegion.get(name);
+    return localMonitoringRegion.get(name.toString());
   }
 
   /**
diff --git a/geode-core/src/main/java/org/apache/geode/redis/internal/RegionProvider.java
b/geode-core/src/main/java/org/apache/geode/redis/internal/RegionProvider.java
index a9d3cfb..8d38c49 100644
--- a/geode-core/src/main/java/org/apache/geode/redis/internal/RegionProvider.java
+++ b/geode-core/src/main/java/org/apache/geode/redis/internal/RegionProvider.java
@@ -214,7 +214,7 @@ public class RegionProvider implements Closeable {
     Region<?, ?> r = this.regions.get(key);
     if (r != null)
       return;
-    if (!this.regions.contains(key)) {
+    if (!this.regions.containsKey(key)) {
       String stringKey = key.toString();
       Lock lock = this.locks.get(stringKey);
       if (lock == null) {


Mime
View raw message