[SPARK-51796] [SQL] Disallow Sort order expressions under non-Sort operators

[mihailoale-db]

What changes were proposed in this pull request?

I propose that we disallow SortOrder expressions under non-Sort operators.

Why are the changes needed?

Following Dataframe program throw an internal error:

val df = sql("SELECT 1 AS name")
df.select(desc("name"))

In this PR we fix is so we throw a more user friendly one.

Does this PR introduce any user-facing change?

Different (more user friendly) error is thrown.

How was this patch tested?

Added test.

Was this patch authored or co-authored using generative AI tooling?

No.

[mihailoale-db]

@cloud-fan should we do it like this or should we move checkExpressionUnderUnsupportedOperator check to the begging of the operator match? Thanks

[cloud-fan]

there are cases that an expression containing SortOrder (e.g. ListAgg) can appear in non-sort operators. We need to find out all these cases.

[github-actions]

We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable.
If you'd like to revive this PR, please reopen it and ask a committer to remove the Stale tag!