rmnskb opened a new pull request, #47977:
URL: https://github.com/apache/arrow/pull/47977

   ### Rationale for this change
   Please see #47966, #47147 assumed that pandas attributes should be JSON 
serializable, whilst they shouldn't be
   
   ### What changes are included in this PR?
   Checked if serializing the pandas attributes raises any errors (`TypeError` 
and `OverflowError`), default to emptying the attributes if that's the case and 
raise a warning that the attributes were emptied.
   
   ### Are these changes tested?
   Yes 
   
   ### Are there any user-facing changes?
   As mentioned above, the users won't be able to attempt to introduce any 
unserializable data to `pd.DataFrame.attrs`
   
   **This PR contains a "Critical Fix" for a bug that causes a crash**, please 
see the issue and the explanation above.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to