Skip to content

ARROW-2454: [C++] Allow zero-array chunked arrays#1897

Closed
pitrou wants to merge 1 commit intoapache:masterfrom
pitrou:ARROW-2454-zero-length-chunked-arrays
Closed

ARROW-2454: [C++] Allow zero-array chunked arrays#1897
pitrou wants to merge 1 commit intoapache:masterfrom
pitrou:ARROW-2454-zero-length-chunked-arrays

Conversation

@pitrou
Copy link
Copy Markdown
Member

@pitrou pitrou commented Apr 16, 2018

This allows code to be more regular and less fragile.

Also fix the chunked array slicing logic.

@pitrou pitrou force-pushed the ARROW-2454-zero-length-chunked-arrays branch from fbbd8a1 to 8ba66df Compare April 16, 2018 10:32
This allows code to be more regular and less fragile.

Also fix the chunked array slicing logic.
@pitrou pitrou force-pushed the ARROW-2454-zero-length-chunked-arrays branch from 8ba66df to 4ad2c6f Compare April 16, 2018 11:26
Copy link
Copy Markdown
Member

@xhochy xhochy left a comment

Choose a reason for hiding this comment

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

+1, LGTM

@pitrou pitrou closed this in 72c7f5d Apr 16, 2018
@pitrou pitrou deleted the ARROW-2454-zero-length-chunked-arrays branch April 16, 2018 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants