Fix PushdownSort dropping LIMIT when eliminating SortExec

[sgrebnov]

Which issue does this PR close?

When PushdownSort removes a SortExec because a source returns Exact (guaranteeing ordering), any fetch (LIMIT) on the SortExec is silently dropped if the underlying plan does not support with_fetch().

For example, ProjectionExec supports try_pushdown_sort (delegating to its child) but does not implement with_fetch(). A plan like SortExec(fetch=10) → ProjectionExec → source that gets sort-eliminated loses the limit.

What changes are included in this PR?

In the Exact branch of PushdownSort, when the eliminated SortExec carried a fetch:

1. Try with_fetch() on the pushed-down source first
2. If with_fetch() returns None, fall back to wrapping with GlobalLimitExec

Are these changes tested?

Yes. Three new unit tests:

• test_sort_pushdown_exact_no_fetch_no_limit — Exact elimination without fetch: no limit wrapper added
• test_sort_pushdown_exact_preserves_fetch_with_global_limit — Exact elimination with fetch, source does NOT support with_fetch(): GlobalLimitExec wrapper added
• test_sort_pushdown_exact_preserves_fetch_with_source_support — Exact elimination with fetch, source supports with_fetch(): limit pushed into source directly

Are there any user-facing changes?

No.

[xudong963]

LGTM, would be better if there are some slt tests

Also, do you want to include it into new minor release?

[alamb]

Thanks @sgrebnov and @martin-g and @xudong963

[sgrebnov]

LGTM, would be better if there are some slt tests

@xudong963 — There's an existing SLT test (sort_pushdown.slt Test 1.3) that already covers the Exact pushdown + LIMIT case where the source supports with_fetch(). The specific fallback path this PR fixes (GlobalLimitExec/LocalLimitExec wrapping when the source doesn't support with_fetch()) can't be triggered through SLT because all built-in data sources support with_fetch(). Adding a custom test-only table provider to the SLT harness just for this seems like overkill, so I'd propose keeping the unit tests with the configurable TestScan. WDYT?

[alamb]

Thanks everyone. If it turns out we can make some slt tests perhaps we can add them as a follow on PR