sungwy commented on code in PR #1632:
URL: https://github.com/apache/iceberg-python/pull/1632#discussion_r2002195447


##########
pyiceberg/types.py:
##########


Review Comment:
   I don't see any type promotions supported to and from `timestamp_ns` and 
`timestamptz_ns` types in the Spec: 
   https://iceberg.apache.org/spec/#primitive-types
   
   And pulled up some issues that have been closed as unplanned:
   - https://github.com/apache/iceberg/issues/9065
   
   And discussion thread where the community decided to vote against 
introducing long -> timestamp_ns conversions:
   https://lists.apache.org/thread/xd7v2znttzvvgx3sqbjf93zn8dwx0v7f



-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to