[jira] [Commented] (GEODE-390) PartitionManager should be removed or deprecated

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

    [ 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 <ji...@pivotal.io>
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 support@gemstone.com before using these APIs
> {noformat}



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