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.

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

[R] argument order in arrow_table affects object return type #35038

Closed
thisisnic opened this issue Apr 11, 2023 · 0 comments · Fixed by #35039
Closed

[R] argument order in arrow_table affects object return type #35038

thisisnic opened this issue Apr 11, 2023 · 0 comments · Fixed by #35039
Assignees
Labels
Breaking Change Includes a breaking change to the API Component: R Type: bug
Milestone

Comments

@thisisnic
Copy link
Member

Describe the bug, including details regarding any error messages, version, and platform.

This is a narrowing of scope from the discussion in #34825.

> class(as.data.frame(arrow::arrow_table(name = "1", mtcars)))
[1] "tbl_df"     "tbl"        "data.frame"
> class(as.data.frame(arrow::arrow_table(mtcars, name = "1")))
[1] "data.frame"

Component(s)

R

@thisisnic thisisnic self-assigned this Apr 11, 2023
thisisnic added a commit that referenced this issue Apr 11, 2023
#35039)

* Closes: #35038

Lead-authored-by: Nic Crane <thisisnic@gmail.com>
Co-authored-by: Neal Richardson <neal.p.richardson@gmail.com>
Signed-off-by: Nic Crane <thisisnic@gmail.com>
@thisisnic thisisnic added this to the 12.0.0 milestone Apr 11, 2023
@wjones127 wjones127 added the Breaking Change Includes a breaking change to the API label Apr 27, 2023
liujiacheng777 pushed a commit to LoongArch-Python/arrow that referenced this issue May 11, 2023
…rn type (apache#35039)

* Closes: apache#35038

Lead-authored-by: Nic Crane <thisisnic@gmail.com>
Co-authored-by: Neal Richardson <neal.p.richardson@gmail.com>
Signed-off-by: Nic Crane <thisisnic@gmail.com>
ArgusLi pushed a commit to Bit-Quill/arrow that referenced this issue May 15, 2023
…rn type (apache#35039)

* Closes: apache#35038

Lead-authored-by: Nic Crane <thisisnic@gmail.com>
Co-authored-by: Neal Richardson <neal.p.richardson@gmail.com>
Signed-off-by: Nic Crane <thisisnic@gmail.com>
rtpsw pushed a commit to rtpsw/arrow that referenced this issue May 16, 2023
…rn type (apache#35039)

* Closes: apache#35038

Lead-authored-by: Nic Crane <thisisnic@gmail.com>
Co-authored-by: Neal Richardson <neal.p.richardson@gmail.com>
Signed-off-by: Nic Crane <thisisnic@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Breaking Change Includes a breaking change to the API Component: R Type: bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants