[GitHub] [arrow] nealrichardson edited a comment on pull request #9950: ARROW-11468: [R] Allow user to pass schema to read_json_arrow()

[GitBox <gi...@apache.org>]

nealrichardson edited a comment on pull request #9950:
URL: https://github.com/apache/arrow/pull/9950#issuecomment-818893574


   > I've added a new test that triggers it but is captured with `expect_error()`.
   > 
   > What do you think, does this make sense? 
   
   If this is expected/documented behavior of the C++ library, I don't think we need a test for it at all in R. 
   
   > 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?
   
   I think that's probably intended, though you could confirm. My guess is that's how the reader will handle named references of nested structs: suppose (or test for yourself) that "third_col" had a value like `{"subcol1": 4, "subcol2": "a"}`, and you mis-typed subcol2, it would probably say "/third_col/subcol2" like a file path. If that's *not* the case, then it seems like a bug. 
   
   


-- 
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:
users@infra.apache.org