Skip to content

Conversation

@sbinet
Copy link
Contributor

@sbinet sbinet commented Sep 2, 2019

No description provided.

@wesm wesm changed the base branch from master to ARROW-6313-flatbuffer-alignment September 2, 2019 14:43
@wesm
Copy link
Member

wesm commented Sep 2, 2019

I just rebased this on the integration branch

Copy link
Member

@wesm wesm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1. The logic looks good to me

wesm pushed a commit that referenced this pull request Sep 5, 2019
Closes #5251 from sbinet/issue-6316 and squashes the following commits:

7427340 <Sebastien Binet> ARROW-6316:  implement new ARROW format with 32b-aligned buffers

Authored-by: Sebastien Binet <binet@cern.ch>
Signed-off-by: Wes McKinney <wesm+git@apache.org>
@wesm wesm closed this Sep 5, 2019
wesm pushed a commit that referenced this pull request Sep 11, 2019
Closes #5251 from sbinet/issue-6316 and squashes the following commits:

7427340 <Sebastien Binet> ARROW-6316:  implement new ARROW format with 32b-aligned buffers

Authored-by: Sebastien Binet <binet@cern.ch>
Signed-off-by: Wes McKinney <wesm+git@apache.org>
@sbinet sbinet deleted the issue-6316 branch September 13, 2019 08:14
wesm pushed a commit that referenced this pull request Sep 13, 2019
Closes #5251 from sbinet/issue-6316 and squashes the following commits:

7427340 <Sebastien Binet> ARROW-6316:  implement new ARROW format with 32b-aligned buffers

Authored-by: Sebastien Binet <binet@cern.ch>
Signed-off-by: Wes McKinney <wesm+git@apache.org>
pprudhvi pushed a commit to pprudhvi/arrow that referenced this pull request Sep 16, 2019
Closes apache#5251 from sbinet/issue-6316 and squashes the following commits:

7427340 <Sebastien Binet> ARROW-6316:  implement new ARROW format with 32b-aligned buffers

Authored-by: Sebastien Binet <binet@cern.ch>
Signed-off-by: Wes McKinney <wesm+git@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants