ndrluis commented on code in PR #2881:
URL: https://github.com/apache/iceberg-python/pull/2881#discussion_r2662752090


##########
pyiceberg/io/pyarrow.py:
##########
@@ -789,7 +789,7 @@ def visit_string(self, _: StringType) -> pa.DataType:
         return pa.large_string()
 
     def visit_uuid(self, _: UUIDType) -> pa.DataType:
-        return pa.uuid()
+        return pa.binary(16)

Review Comment:
   I was referring to this comment 
https://github.com/apache/iceberg-python/pull/2881#discussion_r2662701292, 
which points to the https://github.com/apache/iceberg-python/issues/2372 issue 
where I explain the problem. In this issue your suggestion was to change to 
fixed[16].
   
   @kevinjqliu added the error we're getting now in the other comment. The 
problem is filtering a parquet file with UUID, not reading it.
   ```
   E   pyarrow.lib.ArrowNotImplementedError: Function 'equal' has no kernel 
matching input types (extension<arrow.uuid>, extension<arrow.uuid>)
   
   pyarrow/error.pxi:92: ArrowNotImplementedError
   ```



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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to