Skip to content

Conversation

@cpcloud
Copy link
Contributor

@cpcloud cpcloud commented Feb 3, 2018

No description provided.

Copy link
Member

Choose a reason for hiding this comment

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

Is this call to _column_name_to_strings necessary? I would think that was already done for field_name when you end up here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nope, removed it.

@wesm
Copy link
Member

wesm commented Feb 4, 2018

Any chance anyone has time to look at getting our CI rehabilitated so we can get a passing CI build? #1548 still has some Windows issues

@cpcloud
Copy link
Contributor Author

cpcloud commented Feb 4, 2018

@wesm Looking now.

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, thanks for helping with this @cpcloud! I fixed a flake and rebased so the build should pass now

@wesm
Copy link
Member

wesm commented Feb 5, 2018

Appveyor build is running here: https://ci.appveyor.com/project/cpcloud/arrow/build/1.0.516. Will merge once that is looking good

@wesm wesm closed this in 1a8c9a4 Feb 6, 2018
@simnyatsanga
Copy link
Contributor

@wesm @cpcloud Thanks so much for resolving this issue!

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.

5 participants