DanielLeens commented on PR #12592: URL: https://github.com/apache/seatunnel/pull/12592#issuecomment-6396710656
Thanks @hutiefang76 for reposting the missing details, and thanks @SEZ9 for the focused questions. I checked the unchanged local head: `DuckDBTypeConverter.java:336-397` maps timestamp scale back to native precision, and `DuckDBJdbcRowConverter.java:100-113` warns only after a non-null alias reaches the pre-epoch fallback. Those are implemented behaviors, not promises of a future fix. The metadata-cache pointers and the explicitly remaining Calendar/ResultSet-lifecycle follow-up should stay distinct in the final disposition; there is no separate tracking issue to imply is already open. My source conclusion remains non-blocking, with green PR-side Build as previously recorded. @SEZ9's own final verification and effective maintainer approval remain separate; this is a targeted reply, not another full review or a local test claim. -- 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]
