presto icon indicating copy to clipboard operation
presto copied to clipboard

Warn about global window functions (without partition by)

Open kaikalur opened this issue 1 year ago • 1 comments

They could be slow so good to warn users to instead cross join the result which works better

kaikalur avatar Feb 25 '24 00:02 kaikalur

Or better yet - just rewrite it to be a cross-join (optionally)

kaikalur avatar Feb 25 '24 00:02 kaikalur

I am a student trying to contribute and would like to take work on this issue. Just to make it simple, can you clarify which files need to be modified?

tywe00 avatar Feb 27 '24 13:02 tywe00

I am a student trying to contribute and would like to take work on this issue. Just to make it simple, can you clarify which files need to be modified?

You can look at the analyzer package and expression analyzer. Or you can also do it in the ASTBuilder itself

kaikalur avatar Mar 27 '24 15:03 kaikalur