wesm edited a comment on pull request #6631: URL: https://github.com/apache/arrow/pull/6631#issuecomment-622586463
Problems: * There aren't any unit tests in this patch so there is some work to do to get this merged * Code is duplicated from arrow/util/parsing.h I started working on this so I'll either post an update here or open a new PR. Since there's a good deal more code changes required it would be a good idea to have another round of code review. ---------------------------------------------------------------- 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]
