[GitHub] [iceberg] Fokko commented on pull request #6139: Python: Remove dataclasses

[GitBox <gi...@apache.org>]

Fokko commented on PR #6139:
URL: https://github.com/apache/iceberg/pull/6139#issuecomment-1307812458

   @rdblue added support for wrapping `str` automatically into a `Reference()`. I did notice some funky behavior with the type system. When using a `Reference(UnboundTerm[T])` mypy it is unable to determine the type `T`, since there is no type information. This requires annotating this by hand `Reference[int]("id")`. I think we should only add this type to bound terms.


-- 
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-unsubscribe@iceberg.apache.org

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


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