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

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

GitHub user jinmeiliao opened a pull request:

    https://github.com/apache/incubator-geode/pull/37

    GEODE-390: removing PartitionManager and related tests

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/jinmeiliao/incubator-geode feature/GEODE-390

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-geode/pull/37.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #37
    
----
commit 4a07f458283a493563cbbc9caedade18774a0fae
Author: Jinmei Liao <[email protected]>
Date:   2015-11-18T17:33:28Z

    GEODE-390: removing PartitionManager and related tests

----


> PartitionManager should be removed or deprecated
> ------------------------------------------------
>
>                 Key: GEODE-390
>                 URL: https://issues.apache.org/jira/browse/GEODE-390
>             Project: Geode
>          Issue Type: Wish
>            Reporter: Kirk Lund
>            Assignee: Kirk Lund
>
> Remove or deprecate PartitionManager. This API is broken and was never 
> tested. Given that we do not have an alternative API and this class was never 
> fully supported the first choice it to remove it immediately. But if this can 
> not be done at the very least it should be deprecated.
> ParitionListener should remain in the product but requires unit testing and 
> polishing.
> See GEODE-72:
> {noformat}
> Also consider removing com.gemstone.gemfire.cache.partition.PartitionListener 
> and 
> com.gemstone.gemfire.cache.partition.PartitionManager.
> They have not been deprecated but were never fully supported. Their javadocs 
> say:
>   Note : Please contact [email protected] before using these APIs
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to