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

ASF subversion and git services commented on GEODE-5208:
--------------------------------------------------------

Commit ff662703a48991ed45b2f42ce1ee699142016e4f in geode's branch 
refs/heads/develop from [~gosullivan]
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=ff66270 ]

GEODE-5208 remove test hooks from TypeRegistration (#1958)

GEODE-5208: remove test hooks from TypeRegistration

* Add `@Override` to some methods that didn't have it.
* delete some comments and make some methods less public.
* Remove getClassToType() by checking the same thing another way.


> Remove test hooks from TypeRegistration classes
> -----------------------------------------------
>
>                 Key: GEODE-5208
>                 URL: https://issues.apache.org/jira/browse/GEODE-5208
>             Project: Geode
>          Issue Type: Sub-task
>          Components: membership
>            Reporter: Galen O'Sullivan
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 1h 20m
>  Remaining Estimate: 0h
>
> PeerTypeRegistration has lastAllocatedTypeId and
> lastAllocatedEnumId. These are used on other methods, even put on interfaces, 
> but never actually used by tests. There's also 
> LonerTypeRegistration.testClearRegistry, which is never used.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to