JingsongLi commented on PR #10177: URL: https://github.com/apache/paimon/pull/10177#issuecomment-5831474854
Reviewed fc0af4c for the operator recovery path. The CLI adds clear end-to-end value: it exposes the existing table rollback API after a bad write, and the required mutually exclusive selector prevents an ambiguous target. I checked the CLI dispatch against `FileStoreTable.rollback_to` / `rollback_to_timestamp`, including the REST catalog delegation and error paths. No blocking issue found. Validation: the two new CLI cases passed, including a real two-snapshot table restored to snapshot 1; the broader CLI and rollback selection ran 50 tests successfully. `git diff --check` passed, and CI run 36104718143 is green. The local tests use a filesystem catalog; the existing REST rollback API tests cover its delegate but this PR does not add a live REST CLI test. -- 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]
