[
https://issues.apache.org/jira/browse/HDDS-123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16498546#comment-16498546
]
Bharat Viswanadham edited comment on HDDS-123 at 6/1/18 8:45 PM:
-----------------------------------------------------------------
[~hanishakoneru]
Thank You for the review.
Addressed all the review comments.
Changed the removeContainer, to return boolean, instead of throwing NPE. The
caller can see the return status, and take the decision.
was (Author: bharatviswa):
[~hanishakoneru]
Addressed all the review comments.
Changed the removeContainer, to return boolean, instead of throwing NPE. The
caller can see the return status, and take the decision.
> 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,
> HDDS-123-HDDS-48.02.patch, HDDS-123-HDDS-48.03.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]
