[GitHub] [arrow] hermanschaaf opened a new issue, #35188: [Go] CSV Reader should use AppendValueFromString for extension types

["hermanschaaf (via GitHub)" <gi...@apache.org>]

hermanschaaf opened a new issue, #35188:
URL: https://github.com/apache/arrow/issues/35188

   ### Describe the bug, including details regarding any error messages, version, and platform.
   
   Currently the Go CSV reader uses `dec := json.NewDecoder(strings.NewReader(`"` + str + `"`))`, adding quotes around the string. This works for the tested cases, like UUID, because UUID strings do not contain any additional quotes. However, this approach fails to account for extension types that have quotes that in the string themselves. Instead the recently added `AppendValueFromString` should be used, as this provides the opposite operation for the write which uses `ValueStr`. 
   
   ### Component(s)
   
   Go


-- 
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: issues-unsubscribe@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow] zeroshade closed issue #35188: [Go] CSV Reader should use AppendValueFromString for extension types

["zeroshade (via GitHub)" <gi...@apache.org>]

zeroshade closed issue #35188: [Go] CSV Reader should use AppendValueFromString for extension types
URL: https://github.com/apache/arrow/issues/35188


-- 
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: issues-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org