You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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
The text was updated successfully, but these errors were encountered:
…eader (#35189)
Rather than adding quotes and using `json.NewDecoder` to decode extension types, we should use `AppendValueFromString` which provides the opposite operation to `ValueStr`.
Closes#35188
* Closes: #35188
Authored-by: Herman Schaaf <hermanschaaf@gmail.com>
Signed-off-by: Matt Topol <zotthewizard@gmail.com>
… CSV Reader (apache#35189)
Rather than adding quotes and using `json.NewDecoder` to decode extension types, we should use `AppendValueFromString` which provides the opposite operation to `ValueStr`.
Closesapache#35188
* Closes: apache#35188
Authored-by: Herman Schaaf <hermanschaaf@gmail.com>
Signed-off-by: Matt Topol <zotthewizard@gmail.com>
ArgusLi
pushed a commit
to Bit-Quill/arrow
that referenced
this issue
May 15, 2023
… CSV Reader (apache#35189)
Rather than adding quotes and using `json.NewDecoder` to decode extension types, we should use `AppendValueFromString` which provides the opposite operation to `ValueStr`.
Closesapache#35188
* Closes: apache#35188
Authored-by: Herman Schaaf <hermanschaaf@gmail.com>
Signed-off-by: Matt Topol <zotthewizard@gmail.com>
rtpsw
pushed a commit
to rtpsw/arrow
that referenced
this issue
May 16, 2023
… CSV Reader (apache#35189)
Rather than adding quotes and using `json.NewDecoder` to decode extension types, we should use `AppendValueFromString` which provides the opposite operation to `ValueStr`.
Closesapache#35188
* Closes: apache#35188
Authored-by: Herman Schaaf <hermanschaaf@gmail.com>
Signed-off-by: Matt Topol <zotthewizard@gmail.com>
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 addedAppendValueFromString
should be used, as this provides the opposite operation for the write which usesValueStr
.Component(s)
Go
The text was updated successfully, but these errors were encountered: