ARROW-15665: [C++] Fix error_is_null in strptime with invalid inputs#12732
ARROW-15665: [C++] Fix error_is_null in strptime with invalid inputs#12732rok wants to merge 1 commit intoapache:masterfrom
Conversation
|
@lidavidm I'm a bit embarrassed by this one :) |
lidavidm
left a comment
There was a problem hiding this comment.
Ah, d'oh, we both missed this one. Thanks for fixing this.
|
Benchmark runs are scheduled for baseline = f4dfd6c and contender = 6a0770c. 6a0770c is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
As reported by @dragosmg the following:
Would cause:
This was due to out_data pointer not updating on failed parsing.
This change fixes the issue and adds a test.