Skip to content

Conversation

@wesm
Copy link
Member

@wesm wesm commented Apr 18, 2017

No description provided.

…r time32, time64

Change-Id: I90fc1a710deb6500c8aa5f94a782347b1c931e83
Date64Array
TimestampArray
DecimalArray
ListArray
Copy link
Member Author

Choose a reason for hiding this comment

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

We might consider reorganizing all the logical types by type class instead of by semantic type (e.g. all the List* things together instead of all the Arrays together)

Copy link
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

@asfgit asfgit closed this in 7f20f6e Apr 18, 2017
@wesm wesm deleted the ARROW-818 branch April 18, 2017 15:44
jeffknupp pushed a commit to jeffknupp/arrow that referenced this pull request Jun 3, 2017
…factory functions for time32, time64

Author: Wes McKinney <wes.mckinney@twosigma.com>

Closes apache#557 from wesm/ARROW-818 and squashes the following commits:

96ce436 [Wes McKinney] Expand Sphinx API docs, pyarrow.* namespace. Add factory functions for time32, time64
pribor pushed a commit to GlobalWebIndex/arrow that referenced this pull request Oct 24, 2025
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