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

Colin Patrick McCabe commented on HDFS-5871:
--------------------------------------------

looks good to me.  +1.

> Use PBHelper to serialize CacheDirectiveInfoExpirationProto
> -----------------------------------------------------------
>
>                 Key: HDFS-5871
>                 URL: https://issues.apache.org/jira/browse/HDFS-5871
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>    Affects Versions: HDFS-5698 (FSImage in protobuf)
>            Reporter: Haohui Mai
>            Assignee: Haohui Mai
>         Attachments: HDFS-5871.000.patch, HDFS-5871.001.patch
>
>
> The current implementation of CacheManager serializes the 
> CacheDirectiveInfoExpirationProto in-house. It should call PBHelper.convert() 
>    to avoid code duplication.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to