thisisnic commented on pull request #9950:
URL: https://github.com/apache/arrow/pull/9950#issuecomment-818621376
> > Am I remembering right that we also ran into an issue where attempting
to read a float/integer as a string failed with an odd error message?
>
> I see that now in the PR description. I recommend writing a C++ JIRA for
that, adding a test here that would trigger it, and then `skip("ARROW-12XXX:
Error: Invalid: JSON parse error: Column(/third_col) changed from string to
number in row 0")`
I've had a bit more of a read of the JSON C++ docs and I'm now thinking this
is expected behaviour. The error is only triggered when the values are not
quoted in the JSON, and the table on this page
(https://arrow.apache.org/docs/cpp/json.html#id1) shows that fields of JSON
value type "Number" only have allowed Arrow data types "All Integer types,
Float32, Float64, Date32, Date64, Time32, Time64". I've added a new test that
triggers it but is captured with `expect_error()`.
What do you think, does this make sense? Or even it does, do I still need to
raise a ticket around the formatting of the error with the "/" next to the
column name?
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
