kumarUjjawal commented on PR #22361: URL: https://github.com/apache/datafusion/pull/22361#issuecomment-4494010126
> I'm a little skeptical of the tradeoff here. Obviously planning take >1s for short queries is undesirable, but disabling ALL planning-time evaluation of `regexp_like` seems like we are losing out in the common-case just to avoid a fairly niche corner-case behavior. > > > > What if we tried lowering the regex `size_limit` for regexes that we attempt to evaluate during planning-time, and left more expensive regexes for runtime evaluation? Thanks @neilconway I agree with your scepticism, I had the similar thought while working on this. I will explore this further with your suggestion and see how it goes. -- 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]
