Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[R] Add mention of moving data to/from DuckDB to the vignettes #35709

Closed
thisisnic opened this issue May 22, 2023 · 2 comments · Fixed by #35882
Closed

[R] Add mention of moving data to/from DuckDB to the vignettes #35709

thisisnic opened this issue May 22, 2023 · 2 comments · Fixed by #35882
Assignees
@dgreiss
Labels
Component: R Type: enhancement
Milestone
13.0.0

Comments

@thisisnic
Copy link
Member

Describe the enhancement requested

#35702 documents how to get around the fact we can't do window functions by discussing how to do this with joins, but we should also mention we can do this by passing data to/from DuckDB

Component(s)

R
@dgreiss
Copy link
Contributor

dgreiss commented May 31, 2023

I can take this on if no one else has.

@thisisnic
Copy link
Member Author

Thanks @dgreiss!

@github-actions github-actions bot mentioned this issue Jun 2, 2023
Merged
dgreiss added a commit to dgreiss/arrow that referenced this issue Jun 2, 2023
@dgreiss
Merge branch 'apache:main' into apachegh-35709-doc-updates
fc17eaf
dgreiss added a commit to dgreiss/arrow that referenced this issue Jun 4, 2023
@dgreiss
Revert "Merge branch 'apache:main' into apachegh-35709-doc-updates"
f8b6e9c 
This reverts commit fc17eaf, reversing
changes made to 8a656b1.
thisisnic pushed a commit that referenced this issue Jun 7, 2023
@dgreiss
GH-35709: [R][Documentation] Document passing data to duckdb for wind…
dd26757 
…owed aggregates (#35882)

### Rationale for this change

#35702 documents how to use joins for computing windowed aggregates. This documents an alternative solution by passing data to duckdb. This use case was also mentioned on the [duckdb blog](https://duckdb.org/2021/12/03/duck-arrow.html).

### What changes are included in this PR?

Changes to vignette. 

* Closes: #35709

Authored-by: David Greiss <david.dgreiss@gmail.com>
Signed-off-by: Nic Crane <thisisnic@gmail.com>
@thisisnic thisisnic added this to the 13.0.0 milestone Jun 7, 2023
@dgreiss dgreiss mentioned this issue Jun 11, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dgreiss dgreiss

Labels
Component: R Type: enhancement
Projects
None yet
Milestone
13.0.0
Development

Successfully merging a pull request may close this issue.

GH-35709: [R][Documentation] Document passing data to duckdb for windowed aggregates dgreiss/arrow
2 participants
@dgreiss @thisisnic