[ 
https://issues.apache.org/jira/browse/GEODE-7445?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

John Blum updated GEODE-7445:
-----------------------------
    Component/s: membership

> A public interface method should not return an instance of an "internal" class
> ------------------------------------------------------------------------------
>
>                 Key: GEODE-7445
>                 URL: https://issues.apache.org/jira/browse/GEODE-7445
>             Project: Geode
>          Issue Type: Bug
>          Components: membership
>            Reporter: John Blum
>            Priority: Minor
>
> The {{org.apache.geode.cache.TransactionId}} class's {{getMemberId()}} method 
> returns an instance of 
> {{org.apache.geode.distriburted.internal.InternalDistributedMember}}.
> See here: 
> https://geode.apache.org/releases/latest/javadoc/org/apache/geode/cache/TransactionId.html#getMemberId--
> If anything, the getMemberId methods should return and "ID" and not (even) a 
> {{DistributedMember}}, but, if it must, then it should adhere to the "public" 
> API by return an instance of 
> [{{org.apache.geode.distributed.DistributedMember}}|https://geode.apache.org/releases/latest/javadoc/org/apache/geode/distributed/DistributedMember.html]!



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to