Skip to content

Commit a7bdfca

Browse files
committed
add TODO comment for PyArrow UUID filtering workaround
1 parent d84868d commit a7bdfca

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

pyiceberg/io/pyarrow.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -789,6 +789,10 @@ def visit_string(self, _: StringType) -> pa.DataType:
789789
return pa.large_string()
790790

791791
def visit_uuid(self, _: UUIDType) -> pa.DataType:
792+
# TODO: Change to uuid when PyArrow implements filtering for UUID types
793+
# Using binary(16) instead of pa.uuid() because filtering is not
794+
# implemented for UUID types in PyArrow
795+
# (context: https://github.com/apache/iceberg-python/issues/2372)
792796
return pa.binary(16)
793797

794798
def visit_unknown(self, _: UnknownType) -> pa.DataType:

0 commit comments

Comments
 (0)