-
Notifications
You must be signed in to change notification settings - Fork 3.9k
ARROW-1976: [Python] Handling unicode pandas columns on parquet.read_table #1553
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
python/pyarrow/pandas_compat.py
Outdated
There was a problem hiding this comment.
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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nope, removed it.
|
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 |
|
@wesm Looking now. |
Removing additional instances of using frombytes with str. Removing additional instances of using frombytes with str.
This reverts commit 292ac90.
Change-Id: I28c35f1c32e586a08dd8753dd7de748fa2d3407e
There was a problem hiding this 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
|
Appveyor build is running here: https://ci.appveyor.com/project/cpcloud/arrow/build/1.0.516. Will merge once that is looking good |
No description provided.