taozhiyi13 commented on PR #12738: URL: https://github.com/apache/seatunnel/pull/12738#issuecomment-6142022627
@DanielLeens Thanks for the detailed review. @CryoThrust Thanks for verifying the array path locally. Both points are addressed in `3e4b5bfa`: - Added a multi-row array test in `FragmentConverterTest`. It covers lists of different lengths, an empty list, and a null value. - Updated the English and Chinese docs. They note that the default changed from 1000 to 10000 rows, and that a larger value means fewer fragments but more rows held in memory, so wide-row or vector workloads may need a smaller value. The Map schema mismatch is a pre-existing issue and is tracked in #12745, so I will leave it out of this PR. Fork CI for `3e4b5bfa` is still running. I will update here once the Build check passes. -- 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]
