avantgardnerio opened a new issue, #5046: URL: https://github.com/apache/arrow-datafusion/issues/5046
**Is your feature request related to a problem or challenge? Please describe what you are trying to do.** I've been running into issues related to the SQL values clause: 1. When trying to infer parameter types, the values clause required `param_data_types` which itself had to be inferred 2. When doing insert and update queries, I always get Int64 types, which might need to get coerced into Uint64 for example These issues (as well as several invariants enforced at runtime) seem to be unnecessary if we could pass a Schema down into `value.rs` if one is known, and infer the types being parsed from that Schema - i.e. any time we are doing an `insert into` or can infer placeholder value types. Fundamentally, this issue is questioning [this comment](https://github.com/apache/arrow-datafusion/blob/ab00bc11835f98dd06fa1262d23db2ce1e53a154/datafusion/sql/src/values.rs#L35) and the design around it. **Describe the solution you'd like** Pass an optional schema into the `values` clause parser. **Describe alternatives you've considered** Keep doing it how we are now. @andygrove and @alamb I'd love to hear your thoughts. -- 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]
