goutamadwant commented on PR #12251: URL: https://github.com/apache/seatunnel/pull/12251#issuecomment-5748012127
Addressed in `74defe3d5`: only recognized data-conversion failures follow the skip policies. Unexpected converter failures, unsupported types, incomplete/cyclic cause chains and fatal errors remain non-skippable. Conversion diagnostics and DEBUG skip logs now omit source/extracted values, path expressions and raw causes, while retaining field/type context. A bounded value or attached cause could still expose payloads, so neither is retained. EN/ZH docs now cover row/column precedence, an invalid-int example, unsupported ROUTE_TO_TABLE behavior and the JSONPATH_ERROR_CODE-07 compatibility change. The existing E2E fixture includes invalid-int SKIP. Real nested-row tests use typed configuration; reflection stays local to synthetic fault injection, avoiding a production test hook. The focused conversion suite stays separate from existing path tests. No cause constructor is needed on the new cause-free conversion path; legacy path handling is unchanged. Verified red-before-green: the 51-test run had 25 assertion failures before the fix and now passes with zero failures/errors/skips; reactor verify and Spotless also pass. Docker E2E remains unrun. This does not claim payload sanitization for legacy path errors or unexpected failures. -- 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]
