Re: [I] Remove `numpy` as a dependency [iceberg-python]

2024-10-31 Thread via GitHub
Fokko commented on issue #1259: URL: https://github.com/apache/iceberg-python/issues/1259#issuecomment-2449171063 Hey @corleyma that looks awesome, and that's some amazing speedup 🤤 I think the best place to put this would be Arrow itself. -- This is an automated message from the Apache

Re: [I] Remove `numpy` as a dependency [iceberg-python]

2024-10-30 Thread via GitHub
corleyma commented on issue #1259: URL: https://github.com/apache/iceberg-python/issues/1259#issuecomment-2448429340 Arrow's release cycles are... not quick, but I have no doubt that logic will land there at some point. -- This is an automated message from the Apache Git Service. To resp

Re: [I] Remove `numpy` as a dependency [iceberg-python]

2024-10-30 Thread via GitHub
kevinjqliu commented on issue #1259: URL: https://github.com/apache/iceberg-python/issues/1259#issuecomment-2448411602 supercool! As a project, I don't want to maintain an implementation of the `range` function 😄 Is there a way to contribute `create_arrow_range` to Arrow? -- This is an

Re: [I] Remove `numpy` as a dependency [iceberg-python]

2024-10-30 Thread via GitHub
corleyma commented on issue #1259: URL: https://github.com/apache/iceberg-python/issues/1259#issuecomment-2448393527 > It looks like we're spending a lot of time on generating the sequence. I think it would be good to add this to Arrow itself: https://github.com/apache/arrow/issues/44583 I

Re: [I] Remove `numpy` as a dependency [iceberg-python]

2024-10-30 Thread via GitHub
Fokko closed issue #1259: Remove `numpy` as a dependency URL: https://github.com/apache/iceberg-python/issues/1259 -- 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 unsubscrib

Re: [I] Remove `numpy` as a dependency [iceberg-python]

2024-10-30 Thread via GitHub
Fokko commented on issue #1259: URL: https://github.com/apache/iceberg-python/issues/1259#issuecomment-2446285669 I took the liberty of checking this, so we can fix this for 0.8.0: ![image](https://github.com/user-attachments/assets/1711b206-2ce1-429e-b852-bc6fc2f6c556) It loo

Re: [I] Remove `numpy` as a dependency [iceberg-python]

2024-10-29 Thread via GitHub
Fokko commented on issue #1259: URL: https://github.com/apache/iceberg-python/issues/1259#issuecomment-2444005081 @kevinjqliu That looks about right to me :) Could you do some benchmarking to see if we don't get too much of a performance regression? -- This is an automated message from t