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

George Jahad commented on HDDS-5328:
------------------------------------

I agree "unknown command type" is confusing. Based on the fact that PR #1845 
was closed without merging, I don't think that the Delete Container commend 
will ever exist, and the correct fix for this case is, as you said, to remove 
that command. I will change this ticket/PR, to remove the command. Do you 
agree? Is there someone else I should ask?


In addition, if we do decide to leave it in the cli without adding it to the 
server, I've found another problem.


I fixed the error message here:
https://github.com/GeorgeJahad/ozone/tree/HDDS-5328-fix-delete-container-error-message


But when I test it, it gets retried 300 times, (i think because of the hadoop 
rpc retry mechanism). After the retries, it finally gets to my code to print 
the error, but I doubt users will really wait that long. So if we don't want to 
remove it, I think we may need to fix the retries as well.

> Improve ozone admin container delete command error message for later clients
> ----------------------------------------------------------------------------
>
>                 Key: HDDS-5328
>                 URL: https://issues.apache.org/jira/browse/HDDS-5328
>             Project: Apache Ozone
>          Issue Type: Task
>          Components: Ozone CLI
>            Reporter: Siyao Meng
>            Assignee: George Jahad
>            Priority: Major
>
> Discussion: https://github.com/apache/ozone/pull/1845#issuecomment-770831581
> Keep the command, but should improve the error message to make users feel 
> less like this is a bug by printing sth like: "This command is only available 
> to Ozone SCM version <= X.X. See <link to jira page> for more details."



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to