thswlsqls opened a new pull request, #8612:
URL: https://github.com/apache/paimon/pull/8612
### Purpose
- Several `ActionFactory.printHelp()` messages advertise argument names the
parser does not read; `MultipleParameterToolAdapter` only accepts the exact
snake_case key (or its `_`→`-` variant), so the advertised form is silently
dropped (e.g. `--max_delete` ignored loses the delete cap, `--timeRetained`
ignored keeps a tag forever).
- Corrects 8 factories to match their parser keys and sibling factories:
- `CreateTagFromTimestamp`/`CreateTagFromWatermark`: `--timeRetained` →
`--time_retained`.
- `ExpireSnapshots`/`ExpireChangelogs`: `--max_delete` → `--max_deletes`.
- `ExpirePartitions`: remove phantom `--tag_name` (never parsed by
`create()`).
- `RollbackToTimestamp`: syntax header `rollback_to` →
`rollback_to_timestamp`.
- `RenameTag`: add required `--database`/`--table` (mirrors
`DeleteTagActionFactory`).
- `CreateTag`: add `[--time_retained]` parsed by the parent factory
(mirrors `ReplaceTagActionFactory`).
- Precedent single-sweep help fix: apache/paimon#2440.
### Tests
- Help-text only change, no behavior change — no test added. `mvn
spotless:check` and `checkstyle:check` passed (`-Pflink1`).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
