[
https://issues.apache.org/jira/browse/HDDS-123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16496011#comment-16496011
]
Bharat Viswanadham edited comment on HDDS-123 at 5/31/18 2:58 AM:
------------------------------------------------------------------
Thank You [~msingh] for review.
And also rebased the patch, as now ContainerReportRequestProto, is changed to
ContainerReportsProto. And also removed datanodeDetails, as this is not
required now(ContainerReportsProto does not require datanodeDetails)
Addressed review comments in patch v01.
was (Author: bharatviswa):
Thank You [~msingh] for review.
Addressed review comments in patch v01.
> ContainerSet class to manage ContainerMap
> ------------------------------------------
>
> Key: HDDS-123
> URL: https://issues.apache.org/jira/browse/HDDS-123
> Project: Hadoop Distributed Data Store
> Issue Type: Sub-task
> Reporter: Bharat Viswanadham
> Assignee: Bharat Viswanadham
> Priority: Major
> Fix For: 0.2.1
>
> Attachments: HDDS-123-HDDS-48.00.patch, HDDS-123-HDDS-48.01.patch
>
>
> Create a ContainerSet class, which manages containerMap.
> Previously container map is in ContainerManagerImpl, with refactoring work it
> should be moved to ContainerSet.Â
> This class should handle add/get/remove container from containerMap.
> And also now it should handle containerReport.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
