lidavidm commented on PR #440: URL: https://github.com/apache/arrow-go/pull/440#issuecomment-3071500789
So the expression was bound to the Substrait schema, which always uses String, but the real Arrow data may be LargeString? What I think may be awkward is, doesn't this then mean you can construct and bind an expression that says it evaluates to a String, but then what you actually get back is LargeString? That makes some sense in an abstract type sort of way, but doesn't it just push the type mismatch onto the downstream code calling arrow-go? (Which is probably fine, it lets the caller choose how to handle it.) At least, can we document this in the public API entrypoint? Which seems to be ExecuteScalarExpression. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
