You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "kevinjqliu (via GitHub)" <gi...@apache.org> on 2024/08/11 16:42:40 UTC
[I] [feat] Unify implementation of `to_arrow` and `to_arrow_batch_reader` [iceberg-python]
kevinjqliu opened a new issue, #1039:
URL: https://github.com/apache/iceberg-python/issues/1039
### Feature Request / Improvement
Currently, these two functions are performing similar tasks; the only difference is the output format.
Let's unify the implementations so they don't diverge.
For example, `to_arrow_batch_reader` has `limit` pushed down to the file level, but `to_arrow` does not.
For more details, see https://github.com/apache/iceberg-python/issues/1032#issuecomment-2282819711
--
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-unsubscribe@iceberg.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [I] [feat] Unify implementation of `to_arrow` and `to_arrow_batch_reader` [iceberg-python]
Posted by "sungwy (via GitHub)" <gi...@apache.org>.
sungwy commented on issue #1039:
URL: https://github.com/apache/iceberg-python/issues/1039#issuecomment-2298797090
closed through: https://github.com/apache/iceberg-python/pull/1043
--
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-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [I] [feat] Unify implementation of `to_arrow` and `to_arrow_batch_reader` [iceberg-python]
Posted by "sungwy (via GitHub)" <gi...@apache.org>.
sungwy commented on issue #1039:
URL: https://github.com/apache/iceberg-python/issues/1039#issuecomment-2282945879
Thank you for raising this @kevinjqliu . I think this will be a good improvement to reduce the duplication of code.
Have you started working on this already? If not, would it be alright to get this assigned to myself? 🙂
--
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-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [I] [feat] Unify implementation of `to_arrow` and `to_arrow_batch_reader` [iceberg-python]
Posted by "kevinjqliu (via GitHub)" <gi...@apache.org>.
kevinjqliu commented on issue #1039:
URL: https://github.com/apache/iceberg-python/issues/1039#issuecomment-2282951354
I have not! Please feel free to take a look.
--
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-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [I] [feat] Unify implementation of `to_arrow` and `to_arrow_batch_reader` [iceberg-python]
Posted by "sungwy (via GitHub)" <gi...@apache.org>.
sungwy closed issue #1039: [feat] Unify implementation of `to_arrow` and `to_arrow_batch_reader`
URL: https://github.com/apache/iceberg-python/issues/1039
--
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-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org