I wonder if this should throw an exception instead? I just hate for a user to write "DESC NULLS FIRST" and for us to silently drop the NULLS FIRST portion. The concern is code that consumes the results expecting the nulls first.