pitrou commented on pull request #8312:
URL: https://github.com/apache/arrow/pull/8312#issuecomment-702788372


   > `assert pa.DataType.__str__(ty) == "arrow.py_extension_type<int64>"`
   
   The problem is that this doesn't distinguish between different extension 
types with the same storage type. For example, two extension types representing 
UUIDs and IPv6 addresses, respectively, would get the same string 
representation (because both would be backed by a 16-byte fixed-size binary).
   
   I would be more useful if the string representation looked like 
`extension<arrow.py_extension_type<IntegerType>>`, where `IntegerType` is the 
Python class name.
   
   Optionally this could be made configurable as well.
   


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to