filtering an orderBy causes invalid SQL instead of type error about beam

[tathougies]

I am not seeing this. Your query is correctly generating a subquery on my machine. I've tested on sqlite and postgres. Can you post more context?

What do you mean by 'incorrect SQL'? Did the query not work as expected, or was the syntax rejected by the database?

[3noch]

The syntax was rejected. I'll see if I can repro.

[3noch]

I'm definitely still seeing this with latest master, but perhaps it's more involved. I recently saw it in the query monad style instead:

select $ do
  la <- orderBy_ (\la -> desc_ $ la^.laCreatedUtc)
    $ filter_ (\la -> not_ (la^.laDeleted))
    $ all_ (db^.leadactivity)
  u <- all_ (db^.users)
  guard_ ((la^.laByUser) `references` u)
  pure (la, u)

Moving the orderBy out of the do fixed it.

[tathougies]

@3noch Finally getting around to this. I've managed to reproduce the bug. Looks like projections are being built wrong in a corner case in buildSql92Query. I will investigate and push a fix. Sorry this one took so long!