pepijnve commented on issue #21231: URL: https://github.com/apache/datafusion/issues/21231#issuecomment-4175666286
Ok, I think I see where you're going. I think I might have been thrown off a bit by the term "custom column reference". In your example, you still have to use the builtin `Column` implementation to reference a column. Is it acceptable that we have an API that has that expectation, or would we want to abstract away the usage of `Column` as well? The API you're proposing is in terms of naming scope, the alternative would be in terms of projection where we can extract column indices and rewrite using a projection array. If we leave the recursion up to each `PhysicalExpr` we can achieve the same effect that way. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
