gstvg commented on PR #21679: URL: https://github.com/apache/datafusion/pull/21679#issuecomment-4316819099
Many thanks for you all, @rluvaton @comphead @LiaCastaneda @pepijnve @martin-g. This branch is now up to date. I still must merge from my progressive_lambda_parameters branch, there's a breaking change there, please don't merge this yet. Also, there's two breaking changes from #21172, a new argument o LambdaArgument::evaluate and LambdaVariable field becoming an option, but I will pull only them here instead: add the argument but don't use it, and make field an option that is always Some. I'm working on those right now Finally, I don't like to rush for my own PR's but I do think it's very important that the first 4 major features of #21172 lands on the same release as this, they all are small compared to this one. I would really love to write a blog post, but sometimes I get blocked when writing stuff, I will try my best. And for now I will focus on the 4 major features cited above. > I would advise not to support fixed size lists due to the performance implication for coping valid values I will remove it for now, we can re-add it later if it makes sense -- 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]
