[ 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)