rdblue commented on code in PR #6775: URL: https://github.com/apache/iceberg/pull/6775#discussion_r1209505271
########## python/pyiceberg/io/pyarrow.py: ########## @@ -28,12 +28,16 @@ import os from abc import ABC, abstractmethod from functools import lru_cache, singledispatch +from heapq import merge Review Comment: I think the logic for applying positional deletes in Arrow is about ready. I'd move that to a separate PR so we can get it in before we have the ability to track sequence numbers and do the planning operations. -- 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]
