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

[Go] Add a cgo-based Allocator that does not depend on libarrow #33901

Closed
lidavidm opened this issue Jan 27, 2023 · 0 comments · Fixed by #33902
Closed

[Go] Add a cgo-based Allocator that does not depend on libarrow #33901

lidavidm opened this issue Jan 27, 2023 · 0 comments · Fixed by #33902

Comments

@lidavidm
Copy link
Member

Describe the enhancement requested

As proposed in apache/arrow-go#70. This would let us create buffers that can be safely exported via the C Data Interface.

Component(s)

Go

lidavidm added a commit to lidavidm/arrow that referenced this issue Jan 27, 2023
lidavidm added a commit to lidavidm/arrow that referenced this issue Jan 31, 2023
lidavidm added a commit to lidavidm/arrow that referenced this issue Jan 31, 2023
lidavidm added a commit to lidavidm/arrow that referenced this issue Jan 31, 2023
lidavidm added a commit that referenced this issue Jan 31, 2023
### Rationale for this change

Add a cgo-based allocator for use with the C Data Interface. (See #33900.)

### What changes are included in this PR?

Add a new allocator implementation, `Mallocator`.

### Are these changes tested?

A new test suite was added.

### Are there any user-facing changes?

Adds a new public implementation of an API.
* Closes: #33901

Authored-by: David Li <li.davidm96@gmail.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
@lidavidm lidavidm added this to the 12.0.0 milestone Jan 31, 2023
sjperkins pushed a commit to sjperkins/arrow that referenced this issue Feb 10, 2023
### Rationale for this change

Add a cgo-based allocator for use with the C Data Interface. (See #33900.)

### What changes are included in this PR?

Add a new allocator implementation, `Mallocator`.

### Are these changes tested?

A new test suite was added.

### Are there any user-facing changes?

Adds a new public implementation of an API.
* Closes: apache#33901

Authored-by: David Li <li.davidm96@gmail.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
gringasalpastor pushed a commit to gringasalpastor/arrow that referenced this issue Feb 17, 2023
### Rationale for this change

Add a cgo-based allocator for use with the C Data Interface. (See #33900.)

### What changes are included in this PR?

Add a new allocator implementation, `Mallocator`.

### Are these changes tested?

A new test suite was added.

### Are there any user-facing changes?

Adds a new public implementation of an API.
* Closes: apache#33901

Authored-by: David Li <li.davidm96@gmail.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
fatemehp pushed a commit to fatemehp/arrow that referenced this issue Feb 24, 2023
### Rationale for this change

Add a cgo-based allocator for use with the C Data Interface. (See #33900.)

### What changes are included in this PR?

Add a new allocator implementation, `Mallocator`.

### Are these changes tested?

A new test suite was added.

### Are there any user-facing changes?

Adds a new public implementation of an API.
* Closes: apache#33901

Authored-by: David Li <li.davidm96@gmail.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant