SEZ9 commented on issue #12544: URL: https://github.com/apache/seatunnel/issues/12544#issuecomment-6160503477
Thanks for the detailed proposal. The scope looks reasonable: opt-in via `--debug` / `SEATUNNEL_CLI_DEBUG=1`, default output unchanged, one-line stage summaries with a redacted, truncated detail block on failure, and a short reason code (e.g. `no_hocon_block`) in the soft-failure message. A couple of things that would help before a PR: - Which SeaTunnel version/branch are you working against, and where does this CLI agent pipeline live? That will help find reviewers and confirm which existing secret-redaction logic to reuse. - If possible, share one real redacted example of the current output vs. the desired `--debug` output for a soft-failure case, so the expected format is clear. Since you've indicated you're willing to submit a PR, feel free to go ahead. Please include unit tests covering redaction and truncation of the detail block. @KaustAbhinand thanks for offering to help as well — the issue author has already volunteered to submit a PR, so please coordinate with them here before starting work to avoid duplicate effort. <!-- streview-comment:1747 --> -- 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]
