hutiefang76 commented on issue #12542:
URL: https://github.com/apache/seatunnel/issues/12542#issuecomment-5920024882

   The companion was #12543, but it was closed without merging after checking 
adjacent types. Its native-name mapping changed previously working 
`TIMESTAMP_S`/`TIMESTAMP_MS`/`TIMESTAMP_NS` queries to STRING, and preserving 
unsigned ranges also depends on #12508. I am keeping it withdrawn until those 
cases are addressed; the query fix is not delivered yet.
   
   Yes, STRING is consistent with the existing DuckDB native converter for 
ARRAY/STRUCT/MAP and collection types on the `table_path` path. These are 
driver-provided text values, not nested SeaTunnel values. The source guide's 
current grouping with BLOB/BYTES is inaccurate; the correction is already 
included in #12459, so I have not opened a duplicate docs PR.
   
   A replacement query fix needs the requested native-type coverage plus 
regressions for the previously working timestamp variants and unsigned 
boundaries. Query-based Source validation is still part of that follow-up, not 
something established by this closed PR.


-- 
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]

Reply via email to