westonpace commented on PR #33909: URL: https://github.com/apache/arrow/pull/33909#issuecomment-1423558826
If 2 is acceptable then I would ask that we please move to that. I think the only change is to have the mapper return a declaration instead of a string. If we feel there is still some ambiguity I'd be happy to make a PR / diff real quick to show what I am thinking. -- 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]
