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

Wes McKinney commented on ARROW-8995:
-------------------------------------

Since there is already {{Scalar::ToString}}, let's use that. Does not seem 
justified to have more than one way to format scalar values. 

> [C++] Scalar formatting code used in array/diff.cc should be reusable
> ---------------------------------------------------------------------
>
>                 Key: ARROW-8995
>                 URL: https://issues.apache.org/jira/browse/ARROW-8995
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: C++
>            Reporter: Wes McKinney
>            Priority: Major
>
> Formatting Array values as strings is not specific to the diff.cc code, so it 
> may make sense to move this code elsewhere where it can be used generally 
> (perhaps a method like {{Array::FormatValue}}?). 



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to