You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by "Shashikant Banerjee (Jira)" <ji...@apache.org> on 2020/04/16 11:23:00 UTC
[jira] [Resolved] (HDDS-3387) Fix
ContainerOperationClient#createContainer
[ https://issues.apache.org/jira/browse/HDDS-3387?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Shashikant Banerjee resolved HDDS-3387.
---------------------------------------
Resolution: Fixed
Thanks [~sadanand_shenoy] for the contribution. I have committed this.
> Fix ContainerOperationClient#createContainer
> --------------------------------------------
>
> Key: HDDS-3387
> URL: https://issues.apache.org/jira/browse/HDDS-3387
> Project: Hadoop Distributed Data Store
> Issue Type: Bug
> Affects Versions: 0.6.0
> Reporter: Shashikant Banerjee
> Assignee: Sadanand Shenoy
> Priority: Major
> Labels: pull-request-available
> Fix For: 0.6.0
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> In the following snippet of code , it seems to acquire the xceiverclient instance twice and but release once. It should be acquired only once.
> {code:java}
> @Override
> public ContainerWithPipeline createContainer(HddsProtos.ReplicationType type,
> HddsProtos.ReplicationFactor factor, String owner) throws IOException {
> XceiverClientSpi client = null;
> try {
> // allocate container on SCM.
> ContainerWithPipeline containerWithPipeline =
> storageContainerLocationClient.allocateContainer(type, factor,
> owner);
> Pipeline pipeline = containerWithPipeline.getPipeline();
> client = xceiverClientManager.acquireClient(pipeline);
> // connect to pipeline leader and allocate container on leader datanode.
> client = xceiverClientManager.acquireClient(pipeline);
> createContainer(client,
> {code}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org