You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "Philipp Moritz (JIRA)" <ji...@apache.org> on 2019/02/06 05:35:00 UTC

[jira] [Commented] (ARROW-4491) [Python] Remove usage of std::to_string and std::stoi

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

Philipp Moritz commented on ARROW-4491:
---------------------------------------

Ok, I think I understand this now. On some implementations, int8_t seems to be a typedef to char and the conversion in this case produces a character and not a number.

> [Python] Remove usage of std::to_string and std::stoi
> -----------------------------------------------------
>
>                 Key: ARROW-4491
>                 URL: https://issues.apache.org/jira/browse/ARROW-4491
>             Project: Apache Arrow
>          Issue Type: Improvement
>            Reporter: Philipp Moritz
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> Not sure why this is happening, but for some older compilers I'm seeing
> {code:java}
> terminate called after throwing an instance of 'std::invalid_argument'
>   what():  stoi{code}
> since [https://github.com/apache/arrow/pull/3423|https://github.com/apache/arrow/pull/3423.]
> Possible cause is that there is no int8_t version of [https://en.cppreference.com/w/cpp/string/basic_string/to_string|https://en.cppreference.com/w/cpp/string/basic_string/to_string,] so it might not convert it to a proper string representation of the number.
> Any insight on why this could be happening is appreciated.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)