CryoThrust commented on PR #12101: URL: https://github.com/apache/seatunnel/pull/12101#issuecomment-5578452215
Follow-up on the second review, pushed as a874fdef2:\n\n- Success-result serialization is now outside the validation try/catch. The validation block first builds a ConfigValidationResult, and the output adapter runs only after successful validation; a future serialization failure therefore cannot be reported as a validation failure.\n- The English and Chinese docs now describe the actual stdout contract: the validation result is emitted on stdout, while existing diagnostics may also appear there, so consumers should select the result line. No unrelated logging code was changed.\n\nSpotless apply and git diff --check pass. The remaining Build check is still the GitHub run whose logs/API are unavailable (404); no local production failure has been observed. -- 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]
