[SPARK-33278][SQL][FOLLOWUP] Improve OptimizeWindowFunctions to avoid transfer first to nth_value.

[beliefer]

What changes were proposed in this pull request?

#30178 provided OptimizeWindowFunctions used to transfer first to nth_value.
If the window frame is UNBOUNDED PRECEDING AND CURRENT ROW or UNBOUNDED PRECEDING AND UNBOUNDED FOLLOWING, nth_value has better performance than first.
But the OptimizeWindowFunctions need to exclude other window frame.

Why are the changes needed?

Improve OptimizeWindowFunctions to avoid transfer first to nth_value if the specified window frame isn't UNBOUNDED PRECEDING AND CURRENT ROW or UNBOUNDED PRECEDING AND UNBOUNDED FOLLOWING.

Does this PR introduce any user-facing change?

'No'.

How was this patch tested?

Jenkins test.

[SparkQA]

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35921/

[SparkQA]

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35921/

[SparkQA]

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35922/

[SparkQA]

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35922/

[SparkQA]

Test build #131318 has finished for PR 30419 at commit 6923dd1.

• This patch fails due to an unknown error code, -9.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #131321 has finished for PR 30419 at commit f7d7e7f.

• This patch fails due to an unknown error code, -9.
• This patch merges cleanly.
• This patch adds no public classes.

[beliefer]

retest this please

[SparkQA]

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35939/

[SparkQA]

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35939/

[SparkQA]

Test build #131335 has finished for PR 30419 at commit f7d7e7f.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[beliefer]

cc @cloud-fan

[cloud-fan]

thanks, merging to master!