[ARROW-4746] [C++/Python] PyDataTime_Date wrongly casted to PyDataTime_DateTime

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.15.0
• Component/s: C++, Python
• Labels: pypy
• External issue URL: https://github.com/apache/arrow/issues/16004
• Assignee: Micah Kornfield
• Reporter: Uwe Korn
• Created: 03/Mar/19 12:03
• Updated: 11/Jan/23 07:36
• Resolved: 07/Sep/19 23:14

Description

As mentioned in https://bitbucket.org/pypy/pypy/issues/2842/running-pyarrow-on-pypy-segfaults#comment-50670536, we currently access a PyDataTime_Date object with a PyDataTime_DateTime cast in PyDateTime_DATE_GET_SECOND in our code in two instances. While CPython is able to deal with this wrong usage, PyPy is not able to do so. We should separate the path here into one that deals with dates and another that deals with datetimes.

Reproducible code:

pa.array([datetime.date(2018, 5, 10)], type=pa.date64())

Issue Links

• relates to: ARROW-5125 [Python] Cannot roundtrip extreme dates through pyarrow
• links to: GitHub Pull Request #5306