zeroshade commented on pull request #11359: URL: https://github.com/apache/arrow/pull/11359#issuecomment-961340783
@emkornfield I've addressed the comments you left either via comments or via changes. I tend to lend more towards being less unfriendly wherever possible rather than letting it fail personally which is why I took the tact I did in a lot of the places, but I also see the value in matching behavior of the C++ library in a lot of this since there isn't really a published "spec" for the a JSON form of Arrow it's hard to standardize. Let me know what you think of the changes I've made and comments/documentation I've added. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
