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/04/17 22:57:13 UTC
[GitHub] [arrow] andygrove opened a new pull request #10087: MINOR: [Rust] [Ballista] Ballista now uses relative paths for dependencies
andygrove opened a new pull request #10087:
URL: https://github.com/apache/arrow/pull/10087
This is preperation for moving to the new repo tomorrow
--
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 merged pull request #10087: MINOR: [Rust] [Ballista] Ballista now uses relative paths for dependencies
Posted by GitBox <gi...@apache.org>.
andygrove merged pull request #10087:
URL: https://github.com/apache/arrow/pull/10087
--
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] kszucs commented on pull request #10087: MINOR: [Rust] [Ballista] Ballista now uses relative paths for dependencies
Posted by GitBox <gi...@apache.org>.
kszucs commented on pull request #10087:
URL: https://github.com/apache/arrow/pull/10087#issuecomment-821993064
@andygrove can I include this patch in a new release candidate?
--
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 #10087: MINOR: [Rust] [Ballista] Ballista now uses relative paths for dependencies
Posted by GitBox <gi...@apache.org>.
andygrove commented on pull request #10087:
URL: https://github.com/apache/arrow/pull/10087#issuecomment-821993476
@kszucs Actually, it depends ... if you just patch this PR into the previous RC it might not work. If you are starting fresh with a new RC from master, then yes
--
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] codecov-commenter commented on pull request #10087: MINOR: [Rust] [Ballista] Ballista now uses relative paths for dependencies
Posted by GitBox <gi...@apache.org>.
codecov-commenter commented on pull request #10087:
URL: https://github.com/apache/arrow/pull/10087#issuecomment-821900127
# [Codecov](https://codecov.io/gh/apache/arrow/pull/10087?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
> Merging [#10087](https://codecov.io/gh/apache/arrow/pull/10087?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (26ed136) into [master](https://codecov.io/gh/apache/arrow/commit/27c4fa20a7bc24f53c2d849d21855d8784e1f45f?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (27c4fa2) will **increase** coverage by `0.00%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/arrow/pull/10087?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
```diff
@@ Coverage Diff @@
## master #10087 +/- ##
=======================================
Coverage 78.92% 78.92%
=======================================
Files 286 286
Lines 64717 64717
=======================================
+ Hits 51075 51076 +1
+ Misses 13642 13641 -1
```
| [Impacted Files](https://codecov.io/gh/apache/arrow/pull/10087?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | Coverage Δ | |
|---|---|---|
| [rust/parquet/src/encodings/encoding.rs](https://codecov.io/gh/apache/arrow/pull/10087/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-cnVzdC9wYXJxdWV0L3NyYy9lbmNvZGluZ3MvZW5jb2RpbmcucnM=) | `95.05% <0.00%> (+0.19%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/arrow/pull/10087?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/arrow/pull/10087?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Last update [27c4fa2...26ed136](https://codecov.io/gh/apache/arrow/pull/10087?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
--
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 #10087: MINOR: [Rust] [Ballista] Ballista now uses relative paths for dependencies
Posted by GitBox <gi...@apache.org>.
andygrove commented on pull request #10087:
URL: https://github.com/apache/arrow/pull/10087#issuecomment-821993366
@kszucs yes
--
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] kszucs commented on pull request #10087: MINOR: [Rust] [Ballista] Ballista now uses relative paths for dependencies
Posted by GitBox <gi...@apache.org>.
kszucs commented on pull request #10087:
URL: https://github.com/apache/arrow/pull/10087#issuecomment-822002761
I'm going to cut a new RC from the master after merging #10091
--
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