Skip to content

ARROW-2499: [C++] Factor out Python iteration routines - #1940

Closed
pitrou wants to merge 3 commits into
apache:masterfrom
pitrou:ARROW-2499-python-iteration-refactor
Closed

ARROW-2499: [C++] Factor out Python iteration routines#1940
pitrou wants to merge 3 commits into
apache:masterfrom
pitrou:ARROW-2499-python-iteration-refactor

Conversation

@pitrou

@pitroupitrou commented Apr 24, 2018

Copy link
Copy Markdown
Member

Speeds up list to Arrow conversions by up to 15%. Also fixes a bug where creating a list array would not check that all input items are sequences.

Based on PR #1935.

@pitrou
pitrouforce-pushed the ARROW-2499-python-iteration-refactor branch 3 times, most recently from 23cc975 to 20640f7CompareApril 24, 2018 18:23
@pitroupitrou changed the title [WIP] ARROW-2499: [C++] Factor out Python iteration routinesARROW-2499: [C++] Factor out Python iteration routinesApr 24, 2018
@pitrou
pitrouforce-pushed the ARROW-2499-python-iteration-refactor branch 2 times, most recently from d9fbd37 to 7740c23CompareApril 24, 2018 20:03
@pitrou

Copy link
Copy Markdown
MemberAuthor

There are npm failures on Travis-CI (most probably unrelated, I'd say :-)).

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can you make a JIRA for this and reference as // TODO(ARROW-XXX): ?

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Done.

@pitrou
pitrouforce-pushed the ARROW-2499-python-iteration-refactor branch 2 times, most recently from 0387487 to f2c7e8cCompareApril 30, 2018 21:54

@xhochyxhochy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

A bit of confused by one XXX otherwise this is ready to go

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is this is a problem that should be fixed here?

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

I don't know. Perhaps the naming is simply wrong. I should take a look where it's used.

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Turns out it's unused currently.

@pitrou
pitrouforce-pushed the ARROW-2499-python-iteration-refactor branch from f2c7e8c to f402337CompareMay 1, 2018 11:22

@xhochyxhochy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1, LGTM, merge on green build

@pitrou

pitrou commented May 1, 2018

Copy link
Copy Markdown
MemberAuthor

Looks like PR #1945 needs merging to solve the build issues (or at least some of them).

@pitrou
pitrouforce-pushed the ARROW-2499-python-iteration-refactor branch from f402337 to c93bdcaCompareMay 1, 2018 14:22
@pitrou
pitrouforce-pushed the ARROW-2499-python-iteration-refactor branch from c93bdca to ac31c6cCompareMay 1, 2018 17:00
@pitrou

Copy link
Copy Markdown
MemberAuthor

@pitroupitrou closed this in 2bc110dMay 1, 2018
@pitrou
pitrou deleted the ARROW-2499-python-iteration-refactor branch May 1, 2018 18:56
Sign up for freeto 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

@pitrou@xhochy