[ 
https://issues.apache.org/jira/browse/ARROW-1714?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16225040#comment-16225040
 ] 

ASF GitHub Bot commented on ARROW-1714:
---------------------------------------

wesm commented on a change in pull request #1263: ARROW-1714: [Python] Fix 
invalid serialization/deserialization None name Series
URL: https://github.com/apache/arrow/pull/1263#discussion_r147721200
 
 

 ##########
 File path: python/pyarrow/pandas_compat.py
 ##########
 @@ -450,6 +455,18 @@ def table_to_blockmanager(options, table, memory_pool, 
nthreads=1):
         index = pd.RangeIndex(row_count)
 
     column_strings = [x.name for x in block_table.itercolumns()]
+    if 'columns_metadata' in locals() and columns_metadata is not None:
 
 Review comment:
   I'm not a fan of using `locals()` though I don't know exactly why. I'm going 
to remove this and make `columns_metadata` always None even if there is no 
pandas metadata

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> [Python] No named pd.Series name serialized as u'None'
> ------------------------------------------------------
>
>                 Key: ARROW-1714
>                 URL: https://issues.apache.org/jira/browse/ARROW-1714
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: Python
>    Affects Versions: 0.7.1
>            Reporter: Licht Takeuchi
>            Assignee: Licht Takeuchi
>              Labels: pull-request-available
>             Fix For: 0.8.0
>
>
> Repro code.
> {code:java}
> import pandas as pd
> import pyarrow as pa
> s = pd.Series([1,2,3,4])
> serialized = pa.serialize(s).to_buffer()
> pa.deserialize(serialized).name
> {code}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to