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

[Go] re-enable disabled tests of C Data roundtrip with Python #37339

Closed
pitrou opened this issue Aug 23, 2023 · 2 comments · May be fixed by Dev-Destructor/arrow#1
Closed

[Go] re-enable disabled tests of C Data roundtrip with Python #37339

pitrou opened this issue Aug 23, 2023 · 2 comments · May be fixed by Dev-Destructor/arrow#1
Labels
Component: Go good-first-issue Type: enhancement

Comments

@pitrou
Copy link
Member

pitrou commented Aug 23, 2023

Describe the enhancement requested

This comment can be addressed now that GH-14875 was resolved:

arrow/go/arrow/cdata/test/test_export_to_cgo.py

Lines 165 to 166 in ac5468c

# commented out types can be uncommented after
# GH-14875 is addressed

Component(s)

Go
@pitrou pitrou added Type: enhancement good-first-issue go Pull requests that update Go code and removed go Pull requests that update Go code labels Aug 23, 2023
@pitrou
Copy link
Member Author

pitrou commented Aug 23, 2023

@zeroshade

@Dev-Destructor Dev-Destructor mentioned this issue Aug 25, 2023
Open
@pitrou
Copy link
Member Author

pitrou commented Feb 28, 2024

This seems to have been fixed at some point.

@pitrou pitrou closed this as not planned Won't fix, can't repro, duplicate, stale Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Component: Go good-first-issue Type: enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Commented out pa.run_end_encoded(pa.int16(), pa.int64()), Dev-Destructor/arrow
1 participant
@pitrou