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 2020/08/03 14:50:54 UTC
[GitHub] [arrow] andygrove opened a new pull request #7892: ARROW-9631: [Rust] flight should depend on arrow, not the other way around
andygrove opened a new pull request #7892:
URL: https://github.com/apache/arrow/pull/7892
----------------------------------------------------------------
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] andygrove commented on pull request #7892: ARROW-9631: [Rust] flight should depend on arrow, not the other way around
Posted by GitBox <gi...@apache.org>.
andygrove commented on pull request #7892:
URL: https://github.com/apache/arrow/pull/7892#issuecomment-668328654
@vertexclique I would prefer to have this PR only change the dependency between arrow and flight. This is quite a large change in itself. We can create follow-on PRs to make changes to feature gates.
----------------------------------------------------------------
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 #7892: ARROW-9631: [Rust] flight should depend on arrow, not the other way around
Posted by GitBox <gi...@apache.org>.
nevi-me commented on pull request #7892:
URL: https://github.com/apache/arrow/pull/7892#issuecomment-668495833
> @vertexclique I would prefer to have this PR only change the dependency between arrow and flight. This is quite a large change in itself. We can create follow-on PRs to make changes to feature gates.
or rebase the feature-gating PR against this one when it's merged ...
----------------------------------------------------------------
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] vertexclique commented on pull request #7892: ARROW-9631: [Rust] flight should depend on arrow, not the other way around
Posted by GitBox <gi...@apache.org>.
vertexclique commented on pull request #7892:
URL: https://github.com/apache/arrow/pull/7892#issuecomment-668499855
@nevi-me @andygrove
ok so yeah what i can do moving forward is:
1. create a ticket and rebase master on top.
2. Finalize https://github.com/apache/arrow/pull/7873 to make parquet also use raw arrow.
Sounds good?
----------------------------------------------------------------
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 #7892: ARROW-9631: [Rust] flight should depend on arrow, not the other way around
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #7892:
URL: https://github.com/apache/arrow/pull/7892#issuecomment-668073455
https://issues.apache.org/jira/browse/ARROW-9631
----------------------------------------------------------------
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] vertexclique commented on pull request #7892: ARROW-9631: [Rust] flight should depend on arrow, not the other way around
Posted by GitBox <gi...@apache.org>.
vertexclique commented on pull request #7892:
URL: https://github.com/apache/arrow/pull/7892#issuecomment-668072946
Before merging this, please take a look at mine: https://github.com/apache/arrow/pull/7894
I simplified a bit of what happens when feature gates are enabled.
----------------------------------------------------------------
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] vertexclique edited a comment on pull request #7892: ARROW-9631: [Rust] flight should depend on arrow, not the other way around
Posted by GitBox <gi...@apache.org>.
vertexclique edited a comment on pull request #7892:
URL: https://github.com/apache/arrow/pull/7892#issuecomment-668499855
@nevi-me @andygrove
ok so yeah what I can do moving forward is:
1. create a ticket and rebase master on top.
2. Finalize https://github.com/apache/arrow/pull/7873 to make parquet also use the raw arrow.
Sounds good?
----------------------------------------------------------------
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] andygrove closed pull request #7892: ARROW-9631: [Rust] flight should depend on arrow, not the other way around
Posted by GitBox <gi...@apache.org>.
andygrove closed pull request #7892:
URL: https://github.com/apache/arrow/pull/7892
----------------------------------------------------------------
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