You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "paleolimbot (via GitHub)" <gi...@apache.org> on 2026/09/25 15:04:22 UTC
[PR] perf(r): reuse single-chunk nanoarrow vectors [arrow-nanoarrow]
paleolimbot opened a new pull request, #951:
URL: https://github.com/apache/arrow-nanoarrow/pull/951
Adds an `as_nanoarrow_array.nanoarrow_vctr()` fast path for full-slice, single-chunk vectors. Existing buffers are reused; a shallow array wrapper is created only when a data-frame field name must be attached. This avoids extension round-trips such as rebuilding GeoArrow geometry through WK.
On a 3.23M-row GeoArrow reproduction, data-frame-to-stream conversion fell from 3.44 s to 0.32 s. The available R test suite and `R CMD check --no-manual` pass.
```r
library(nanoarrow)
x <- as_nanoarrow_vctr(as_nanoarrow_array(c("one", "two")))
as.data.frame(as_nanoarrow_array(data.frame(x = x)))
#> x
#> 1 one
#> 2 two
```
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix(r): reuse single-chunk nanoarrow vectors in as_nanoarrow_array [arrow-nanoarrow]
Posted by "paleolimbot (via GitHub)" <gi...@apache.org>.
paleolimbot merged PR #951:
URL: https://github.com/apache/arrow-nanoarrow/pull/951
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] perf(r): reuse single-chunk nanoarrow vectors [arrow-nanoarrow]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #951:
URL: https://github.com/apache/arrow-nanoarrow/pull/951#issuecomment-5834716811
## [Codecov](https://app.codecov.io/gh/apache/arrow-nanoarrow/pull/951?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
:x: Patch coverage is `92.00000%` with `2 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 78.21%. Comparing base ([`8f81815`](https://app.codecov.io/gh/apache/arrow-nanoarrow/commit/8f81815c7a233d3fc697f43c7aed42e82a4f33be?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`cc80c4f`](https://app.codecov.io/gh/apache/arrow-nanoarrow/commit/cc80c4f54fedab1449159ebb7b9e4caad850277c?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/arrow-nanoarrow/pull/951?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [r/R/vctr.R](https://app.codecov.io/gh/apache/arrow-nanoarrow/pull/951?src=pr&el=tree&filepath=r%2FR%2Fvctr.R&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ci9SL3ZjdHIuUg==) | 92.00% | [2 Missing :warning: ](https://app.codecov.io/gh/apache/arrow-nanoarrow/pull/951?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## main #951 +/- ##
==========================================
+ Coverage 78.19% 78.21% +0.02%
==========================================
Files 106 106
Lines 16845 16870 +25
Branches 1985 1985
==========================================
+ Hits 13172 13195 +23
- Misses 2439 2441 +2
Partials 1234 1234
```
</details>
[:umbrella: View full report in Codecov by Harness](https://app.codecov.io/gh/apache/arrow-nanoarrow/pull/951?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
:loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
<details><summary> :rocket: New features to boost your workflow: </summary>
- :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems.
</details>
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org