You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2021/01/07 08:59:42 UTC
[GitHub] [arrow] nevi-me opened a new pull request #9123: ARROW-10917: [Doc] Update feature matrix for Rust
nevi-me opened a new pull request #9123:
URL: https://github.com/apache/arrow/pull/9123
Updates matrix for work done since `2.0.0`
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] mqy commented on pull request #9123: ARROW-10917: [Doc] Update feature matrix for Rust
Posted by GitBox <gi...@apache.org>.
mqy commented on pull request #9123:
URL: https://github.com/apache/arrow/pull/9123#issuecomment-756132067
The custom metadata is not implemented for in file footer, dictionary and record batch, should we add this into 3.0.0?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] mqy edited a comment on pull request #9123: ARROW-10917: [Doc] Update feature matrix for Rust
Posted by GitBox <gi...@apache.org>.
mqy edited a comment on pull request #9123:
URL: https://github.com/apache/arrow/pull/9123#issuecomment-756148567
> Yes please, I thought it was implemented. Or do you mean in the IPC format? I'll untick it in the meantime
I mean the IPC format, I'll push some incomplete changes soon.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] github-actions[bot] commented on pull request #9123: ARROW-10917: [Doc] Update feature matrix for Rust
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #9123:
URL: https://github.com/apache/arrow/pull/9123#issuecomment-755981413
https://issues.apache.org/jira/browse/ARROW-10917
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] pitrou commented on a change in pull request #9123: ARROW-10917: [Doc] Update feature matrix for Rust
Posted by GitBox <gi...@apache.org>.
pitrou commented on a change in pull request #9123:
URL: https://github.com/apache/arrow/pull/9123#discussion_r553345704
##########
File path: docs/source/status.rst
##########
@@ -161,6 +161,7 @@ Notes:
* \(1) No support for handshake or DoExchange.
* \(2) Support using AspNetCore authentication handlers.
+* \(3) Flight integration testing for Rust is still ongoing as of version 3.0.0, see ARROW-8853.
Review comment:
I don't think we should add information about work-in-progress features here. This page is for end-users.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] pitrou commented on pull request #9123: ARROW-10917: [Doc] Update feature matrix for Rust
Posted by GitBox <gi...@apache.org>.
pitrou commented on pull request #9123:
URL: https://github.com/apache/arrow/pull/9123#issuecomment-758836570
Green CI on my fork: https://github.com/pitrou/arrow/actions/runs/480864596
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] pitrou commented on pull request #9123: ARROW-10917: [Doc] Update feature matrix for Rust
Posted by GitBox <gi...@apache.org>.
pitrou commented on pull request #9123:
URL: https://github.com/apache/arrow/pull/9123#issuecomment-758821003
I've taken the liberty of updating this PR. Will merge if CI green.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] nevi-me commented on a change in pull request #9123: ARROW-10917: [Doc] Update feature matrix for Rust
Posted by GitBox <gi...@apache.org>.
nevi-me commented on a change in pull request #9123:
URL: https://github.com/apache/arrow/pull/9123#discussion_r553191590
##########
File path: docs/source/status.rst
##########
@@ -169,18 +170,18 @@ Notes:
C Data Interface
================
-+-----------------------------+-------+--------+-------+
-| Feature | C++ | Python | R |
-| | | | |
-+=============================+=======+========+=======+
-| Schema export | ✓ | ✓ | ✓ |
-+-----------------------------+-------+--------+-------+
-| Array export | ✓ | ✓ | ✓ |
-+-----------------------------+-------+--------+-------+
-| Schema import | ✓ | ✓ | ✓ |
-+-----------------------------+-------+--------+-------+
-| Array import | ✓ | ✓ | ✓ |
-+-----------------------------+-------+--------+-------+
++-----------------------------+-------+--------+-------+-------+
Review comment:
@jorgecarleitao this one's your baby ❤️, may you please populate what's supported on the C Data interface in Rust.
Please push directly on my branch
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] alamb commented on pull request #9123: ARROW-10917: [Doc] Update feature matrix for Rust
Posted by GitBox <gi...@apache.org>.
alamb commented on pull request #9123:
URL: https://github.com/apache/arrow/pull/9123#issuecomment-758265481
@nevi-me is this ready to go? I think the 3.0 release is just about to happen so we should probably merge it in soon (TM)
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] mqy commented on pull request #9123: ARROW-10917: [Doc] Update feature matrix for Rust
Posted by GitBox <gi...@apache.org>.
mqy commented on pull request #9123:
URL: https://github.com/apache/arrow/pull/9123#issuecomment-756148567
> > The custom metadata is not implemented for file footer, dictionary and record batch, should we add this into 3.0.0?
>
> Yes please, I thought it was implemented. Or do you mean in the IPC format? I'll untick it in the meantime
I mean the IPC format, I'll push some incomplete changes soon.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] mqy edited a comment on pull request #9123: ARROW-10917: [Doc] Update feature matrix for Rust
Posted by GitBox <gi...@apache.org>.
mqy edited a comment on pull request #9123:
URL: https://github.com/apache/arrow/pull/9123#issuecomment-756132067
The custom metadata is not implemented for file footer, dictionary and record batch, should we add this into 3.0.0?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] pitrou closed pull request #9123: ARROW-10917: [Doc] Update feature matrix for Rust
Posted by GitBox <gi...@apache.org>.
pitrou closed pull request #9123:
URL: https://github.com/apache/arrow/pull/9123
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] nevi-me commented on pull request #9123: ARROW-10917: [Doc] Update feature matrix for Rust
Posted by GitBox <gi...@apache.org>.
nevi-me commented on pull request #9123:
URL: https://github.com/apache/arrow/pull/9123#issuecomment-756140755
> The custom metadata is not implemented for file footer, dictionary and record batch, should we add this into 3.0.0?
Yes please, I thought it was implemented. Or do you mean in the IPC format? I'll untick it in the meantime
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] nevi-me commented on a change in pull request #9123: ARROW-10917: [Doc] Update feature matrix for Rust
Posted by GitBox <gi...@apache.org>.
nevi-me commented on a change in pull request #9123:
URL: https://github.com/apache/arrow/pull/9123#discussion_r553350814
##########
File path: docs/source/status.rst
##########
@@ -161,6 +161,7 @@ Notes:
* \(1) No support for handshake or DoExchange.
* \(2) Support using AspNetCore authentication handlers.
+* \(3) Flight integration testing for Rust is still ongoing as of version 3.0.0, see ARROW-8853.
Review comment:
Thanks, I'll remove WIP
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] nevi-me commented on a change in pull request #9123: ARROW-10917: [Doc] Update feature matrix for Rust
Posted by GitBox <gi...@apache.org>.
nevi-me commented on a change in pull request #9123:
URL: https://github.com/apache/arrow/pull/9123#discussion_r553191933
##########
File path: docs/source/status.rst
##########
@@ -161,6 +161,7 @@ Notes:
* \(1) No support for handshake or DoExchange.
* \(2) Support using AspNetCore authentication handlers.
+* \(3) Flight integration testing for Rust is still ongoing as of version 3.0.0, see ARROW-8853.
Review comment:
@carols10cents I haven't ticked anything here, as we haven't merged the flight test PR
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] alamb commented on pull request #9123: ARROW-10917: [Doc] Update feature matrix for Rust
Posted by GitBox <gi...@apache.org>.
alamb commented on pull request #9123:
URL: https://github.com/apache/arrow/pull/9123#issuecomment-758826510
Thank you @pitrou
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org