dingo4dev commented on issue #2209: URL: https://github.com/apache/iceberg-python/issues/2209#issuecomment-3072643534
I've go thought some popular repos which also depends on pyarrow. Using if else condition to check version might be the simple and easy way to branch pyarrow version. The downside is the function may have masses code regarding to different version. Using decorate dispatch is good for readability but I not sure is it worth to add these which will increase the maintenance process > Dask > https://github.com/dask/dask/blob/3f6051ae242f894af0f43fd9de7ad7131fb3e70d/dask/dataframe/_compat.py#L27-L28 > Pandas > https://github.com/pandas-dev/pandas/blob/bc6ad140daf230c470fef92bec598831d4f94a16/pandas/compat/pyarrow.py#L17 -- 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-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
