You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@reef.apache.org by "Markus Weimer (JIRA)" <ji...@apache.org> on 2015/01/23 18:17:35 UTC

[jira] [Comment Edited] (REEF-41) Replace Guava cache usage in core REEF

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

Markus Weimer edited comment on REEF-41 at 1/23/15 5:17 PM:
------------------------------------------------------------

I did a quick scan of the code. Looks good, but it would be good to add JavaDoc to the tests. 

I won't be able to do a deep code review until Monday. It might be good for someone else to do that.


was (Author: markus.weimer):
I did a quick scan of the code. Looks good, but it would be good to add JavaDoc to the tests. 

> Replace Guava cache usage in core REEF
> --------------------------------------
>
>                 Key: REEF-41
>                 URL: https://issues.apache.org/jira/browse/REEF-41
>             Project: REEF
>          Issue Type: Sub-task
>          Components: REEF
>            Reporter: Brian Cho
>            Assignee: Brian Cho
>
> This sub-Task will deal with removing the non-trivial instance of Guava usage. Specifically:
> ( quoting [~bgchun] )
> NameCache uses a Guava cache. One should be able to cleanly replace it by
> providing a Cache<Identifier, InetSocketAddress> implementation. But this
> requires more effort since NameCache relies on a few neat features of the
> Guava cache.



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