]> granicus.if.org Git - postgresql/commit
Check misplaced window functions before checking aggregate/group by sanity.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 8 Feb 2012 18:15:02 +0000 (13:15 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 8 Feb 2012 18:15:02 +0000 (13:15 -0500)
commitcb7c84fae8a6780d836687aa2c9655eb936ebd25
tree9411d1899e8b2342f03e973a9cd472fd30dd02c8
parentcbba55d6d792b55f6b448a31fc14aef84510967c
Check misplaced window functions before checking aggregate/group by sanity.

If somebody puts a window function in WHERE, we should complain about that
in so many words.  The previous coding tended to complain about the window
function's arguments instead, which is likely to be misleading to users who
are unclear on the semantics of window functions; as seen for example in
bug #6440 from Matyas Novak.

Just another example of how "add new code at the end" is frequently a bad
heuristic.
src/backend/parser/analyze.c