jayzhan211 commented on PR #9504: URL: https://github.com/apache/arrow-datafusion/pull/9504#issuecomment-1986714975
> 1. Make sure we have `Scalar` that we need `make_scalar_function_with_hints`, since I think we should only get `Array` for array functions, if we really have `Scalar`, it seems something wrong to me. I think we need `make_scalar_function_with_hints`, Scalar::List are represented in ColumnValue::Scalar, and others are represented in ColumnValue::Array. Although we don't do the optimized Scalar path for array functions, to be consistent with the definition of ColumnValue, it may be nice be convert it to ColumnValue::Scalar if we got Scalar::List. -- 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]
