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

Bharat Viswanadham commented on HDDS-1024:
------------------------------------------

Hi [~linyiqun]

Thanks for the review.

Attached the patch to address other review comments.

Reason for not using GenericTestUtils.waitFor, is because we check whether it 
is in containerset or not, it will immediately return true. (As we are not sure 
whether this command is processed by datanode or not, so waiting for 10 seconds 
to be sure command must have been processed by datanode. This is the reason for 
not using GenericTestUtils.waitFor).  

> Handle DeleteContainerCommand in the SCMDatanodeProtocolServer
> --------------------------------------------------------------
>
>                 Key: HDDS-1024
>                 URL: https://issues.apache.org/jira/browse/HDDS-1024
>             Project: Hadoop Distributed Data Store
>          Issue Type: Bug
>            Reporter: Bharat Viswanadham
>            Assignee: Bharat Viswanadham
>            Priority: Major
>              Labels: SCM
>         Attachments: HDDS-1024.00.patch, HDDS-1024.01.patch, 
> HDDS-1024.02.patch, HDDS-1024.03.patch, HDDS-1024.04.patch
>
>
> Right now, in the SCMDatanodeProtocolServer getCommandResponse() 
> deleteContainerCommand is not handled, so deleteContainerCommand is not sent 
> to Datanode.
>  
> The deletecontainercommand request is sent for over replicated containers, so 
> this over replication is currently broken because of this.
>  
> Because of this we see below error:
>  
> {code:java}
> java.lang.IllegalArgumentException: Not implemented
>  at 
> org.apache.hadoop.hdds.scm.server.SCMDatanodeProtocolServer.getCommandResponse(SCMDatanodeProtocolServer.java:345)
>  at 
> org.apache.hadoop.hdds.scm.server.SCMDatanodeProtocolServer.sendHeartbeat(SCMDatanodeProtocolServer.java:272)
>  at 
> org.apache.hadoop.ozone.protocolPB.StorageContainerDatanodeProtocolServerSideTranslatorPB.sendHeartbeat(StorageContainerDatanodeProtocolServerSideTranslatorPB.java:88)
>  at 
> org.apache.hadoop.hdds.protocol.proto.StorageContainerDatanodeProtocolProtos$StorageContainerDatanodeProtocolService$2.callBlockingMethod(StorageContainerDatanodeProtocolProtos.java:27753)
>  at 
> org.apache.hadoop.ipc.ProtobufRpcEngine$Server$ProtoBufRpcInvoker.call(ProtobufRpcEngine.java:524)
>  at org.apache.hadoop.ipc.RPC$Server.call(RPC.java:1025)
>  at org.apache.hadoop.ipc.Server$RpcCall.run(Server.java:876)
>  at org.apache.hadoop.ipc.Server$RpcCall.run(Server.java:822)
>  at java.security.AccessController.doPrivileged(Native Method)
>  at javax.security.auth.Subject.doAs(Subject.java:422)
>  at 
> org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1730)
>  at org.apache.hadoop.ipc.Server$Handler.run(Server.java:2682)
> {code}
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to