kmitchener commented on PR #3309: URL: https://github.com/apache/arrow-datafusion/pull/3309#issuecomment-1233177982
@alamb @andygrove what do you think about adding unsigned int support per #3307 here vs. a separate PR? Also, in general is unsigned int support via SQL desired? Also, note that I removed what was almost a duplicate function in planner.rs -- now it will use the same function to convert SQL datatypes to Arrow datatypes for both SQL CAST() function and for the schema that can be specified for CREATE EXTERNAL TABLE. I'd just like to verify with someone more knowledgeable than me that this is a good change. -- 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]
