Skip to content

ARROW-717: [C++] Implement IPC zero-copy round trip for tensors - #454

Closed
wesm wants to merge 1 commit into
apache:masterfrom
wesm:ARROW-717
Closed

ARROW-717: [C++] Implement IPC zero-copy round trip for tensors#454
wesm wants to merge 1 commit into
apache:masterfrom
wesm:ARROW-717

Conversation

@wesm

@wesmwesm commented Mar 30, 2017•
edited
Loading

Copy link
Copy Markdown
Member

This patch provides:

WriteTensor(tensor, file, &metadata_length, &body_length));
std::shared_ptr<Tensor>result;
ReadTensor(offset, file, &result));

Also implemented Tensor::Equals and did some refactoring / code simplification in compare.cc

@wesm

wesm commented Mar 30, 2017

Copy link
Copy Markdown
MemberAuthor

green build: https://travis-ci.org/wesm/arrow/builds/216839036

@wesm

wesm commented Mar 30, 2017

Copy link
Copy Markdown
MemberAuthor

rebased

@wesm
Tensor IPC read/write, and refactoring / code scrubbing
6c15481
Change-Id: Ib0d982639ef3d94a4844656bd56c5e62a830b18e
@wesm

wesm commented Mar 30, 2017

Copy link
Copy Markdown
MemberAuthor

rebased. merging on green build

@wesm

wesm commented Mar 30, 2017

Copy link
Copy Markdown
MemberAuthor

Here's the appveyor build: https://ci.appveyor.com/project/wesm/arrow/build/1.0.32

@wesm
wesm deleted the ARROW-717 branch March 30, 2017 22:03
wesm pushed a commit to wesm/arrow that referenced this pull request Sep 8, 2018
@pitrou@xhochy
PARQUET-1268: Fix conversion of null list Arrow arrays
a251714
Author: Antoine Pitrou <antoine@python.org>
Closesapache#454 from pitrou/PARQUET-1268-null-list-conversion and squashes the following commits:
e4e7744 [Antoine Pitrou] PARQUET-1268: Fix conversion of null list Arrow arrays
Change-Id: Ie43bbdd282cd691e641412782f11e1dc651e730d
pribor pushed a commit to GlobalWebIndex/arrow that referenced this pull request Oct 24, 2025
@ViggoC
apacheGH-48: Implement VectorAppender for BaseVariableWidthViewVector (…
db8b6aa
…apache#454)
Fixesapache#48.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xhochyxhochyxhochy approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@wesm@xhochy