[ 
https://issues.apache.org/jira/browse/HUDI-5804?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Pramod Biligiri updated HUDI-5804:
----------------------------------
    Description: 
In multiple places in the CommitsCommand, the ShellOption is missing the "–" 
parameter in its value attribute. One such example is shown below from "commit 
showpartitions":

[https://github.com/apache/hudi/blob/master/hudi-cli/src/main/java/org/apache/hudi/cli/commands/CommitsCommand.java#L213]
|@ShellOption(value = \{"includeArchivedTimeline"}, help = "Include archived 
commits as well", defaultValue = "false") final boolean 
includeArchivedTimeline)|

In the above, it should read 'value=\{"--includeArchivedTimeline"...}'

 

 

  was:
In multiple places in the CommitsCommand, the ShellOption is missing the "–" 
parameter in its value attribute. One such example is shown below from "commit 
showpartitions":

[https://github.com/apache/hudi/blob/master/hudi-cli/src/main/java/org/apache/hudi/cli/commands/CommitsCommand.java#L213]
|@ShellOption(value = {"includeArchivedTimeline"}, help = "Include archived 
commits as well", defaultValue = "false") final boolean 
includeArchivedTimeline)|


That should read value=\{"--includeArchivedTimeline"}

 

 


> hudi-cli CommitsCommand - some options fail due to typo in ShellOption 
> annotation
> ---------------------------------------------------------------------------------
>
>                 Key: HUDI-5804
>                 URL: https://issues.apache.org/jira/browse/HUDI-5804
>             Project: Apache Hudi
>          Issue Type: Bug
>          Components: cli
>            Reporter: Pramod Biligiri
>            Priority: Minor
>
> In multiple places in the CommitsCommand, the ShellOption is missing the "–" 
> parameter in its value attribute. One such example is shown below from 
> "commit showpartitions":
> [https://github.com/apache/hudi/blob/master/hudi-cli/src/main/java/org/apache/hudi/cli/commands/CommitsCommand.java#L213]
> |@ShellOption(value = \{"includeArchivedTimeline"}, help = "Include archived 
> commits as well", defaultValue = "false") final boolean 
> includeArchivedTimeline)|
> In the above, it should read 'value=\{"--includeArchivedTimeline"...}'
>  
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to