[ https://issues.apache.org/jira/browse/GIRAPH-85?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13217096#comment-13217096 ]
Avery Ching commented on GIRAPH-85: ----------------------------------- I just looked at your patch in Eclipse and see that there is now a warning due to Type safety: Unchecked cast from VersionedProtocol to CommunicationsInterface<I,V,E,M>. We can either keep this the way it was before, or add @SuppressWarnings("unchecked") to the method. I don't have a strong opinion here. Anyone else care to comment? > Simplify return expression in RPCCommunications::getRPCProxy > ------------------------------------------------------------ > > Key: GIRAPH-85 > URL: https://issues.apache.org/jira/browse/GIRAPH-85 > Project: Giraph > Issue Type: Improvement > Affects Versions: 0.2.0 > Reporter: Jakob Homan > Labels: newbie > Fix For: 0.2.0 > > Attachments: GIRAPH-85.patch, GIRAPH-85.patch > > > Twice in RPCCommunications::getRPCProxy a local variable, proxy, is created > and immediately returned. We can simplify this to just return the value. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira