geode-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (GEODE-254) Remove deprecated Region.keys and Region.entries
Date Mon, 08 May 2017 19:29:04 GMT

    [ https://issues.apache.org/jira/browse/GEODE-254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16001397#comment-16001397
] 

ASF GitHub Bot commented on GEODE-254:
--------------------------------------

Github user dschneider-pivotal commented on a diff in the pull request:

    https://github.com/apache/geode/pull/488#discussion_r115331529
  
    --- Diff: geode-core/src/main/java/org/apache/geode/cache/client/internal/ProxyRegion.java
---
    @@ -388,7 +388,7 @@ public Set keySetOnServer() {
       public Set keys() {
    --- End diff --
    
    I think you want this old "keys()" implementation to now be "keySet()". The old "keySet"
did not properly call preOp/postOp.


> Remove deprecated Region.keys and Region.entries
> ------------------------------------------------
>
>                 Key: GEODE-254
>                 URL: https://issues.apache.org/jira/browse/GEODE-254
>             Project: Geode
>          Issue Type: Sub-task
>            Reporter: Darrel Schneider
>            Assignee: Avinash Dongre
>             Fix For: 1.2.0
>
>   Original Estimate: 2h
>  Remaining Estimate: 2h
>
> Remove the deprecated Region.keys and Region.entries. Any calls can be simply changed
to Region.keySet and Region.entrySet so this should be an easy change.
> A large number of tests call the deprecated methods.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Mime
View raw message