SEZ9 commented on issue #12507: URL: https://github.com/apache/seatunnel/issues/12507#issuecomment-5862579362
Thanks for the clear report and boundary table. Since you already have a fix and a regression covering the upper boundaries, zero, null, and round trips into precreated and catalog-generated tables, please open a PR referencing this issue. Widening the unsigned mappings to SMALLINT/INT/BIGINT/DECIMAL(20,0) to match the documented DuckDB source mapping sounds like the right direction, and keeping the signed integer mappings unchanged is good. Could you also confirm which SeaTunnel version you reproduced this on? That will help decide whether the fix should be backported. <!-- streview-comment:1374 --> -- 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]
