refactor(proto): migrate window serde

[Phoenix500526]

Which issue does this PR close?

• Closes Proto: migrate WindowAggExec + BoundedWindowAggExec #23513.

Rationale for this change

Window plans still relied on centralized protobuf dispatch, keeping serialization separate from the execution plans that own their state. Both window executors share one protobuf variant. Decoding must inspect input_order_mode before selecting the concrete plan.

Add plan-local encoders for both executors and a decoder for the shared node. Keep the legacy helpers as deprecated delegates while preserving window frames and UDF payloads on the existing wire format.

What changes are included in this PR?

Are these changes tested?

Yes

Are there any user-facing changes?

[github-actions]

Thank you for opening this pull request!

Reviewer note: cargo-semver-checks reported the current version number is not SemVer-compatible with the changes in this pull request (compared against the base branch).

Details

     Cloning apache/main
    Building datafusion-physical-plan v54.1.0 (current)
       Built [  42.593s] (current)
     Parsing datafusion-physical-plan v54.1.0 (current)
      Parsed [   0.141s] (current)
    Building datafusion-physical-plan v54.1.0 (baseline)
       Built [  37.196s] (baseline)
     Parsing datafusion-physical-plan v54.1.0 (baseline)
      Parsed [   0.141s] (baseline)
    Checking datafusion-physical-plan v54.1.0 -> v54.1.0 (no change; assume patch)
     Checked [   0.621s] 223 checks: 223 pass, 30 skip
     Summary no semver update required
    Finished [  82.027s] datafusion-physical-plan
    Building datafusion-proto v54.1.0 (current)
       Built [  60.470s] (current)
     Parsing datafusion-proto v54.1.0 (current)
      Parsed [   0.019s] (current)
    Building datafusion-proto v54.1.0 (baseline)
       Built [  60.579s] (baseline)
     Parsing datafusion-proto v54.1.0 (baseline)
      Parsed [   0.020s] (baseline)
    Checking datafusion-proto v54.1.0 -> v54.1.0 (no change; assume patch)
     Checked [   0.232s] 223 checks: 222 pass, 1 fail, 0 warn, 30 skip

--- failure trait_method_marked_deprecated: trait method #[deprecated] added ---

Description:
A trait method is now #[deprecated]. Downstream crates will get a compiler warning when using this method.
        ref: https://doc.rust-lang.org/reference/attributes/diagnostics.html#the-deprecated-attribute
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.49.0/src/lints/trait_method_marked_deprecated.ron

Failed in:
  method try_into_window_physical_plan in trait datafusion_proto::physical_plan::PhysicalPlanNodeExt in /home/runner/work/datafusion/datafusion/datafusion/proto/src/physical_plan/mod.rs:713
  method try_from_window_agg_exec in trait datafusion_proto::physical_plan::PhysicalPlanNodeExt in /home/runner/work/datafusion/datafusion/datafusion/proto/src/physical_plan/mod.rs:713
  method try_from_bounded_window_agg_exec in trait datafusion_proto::physical_plan::PhysicalPlanNodeExt in /home/runner/work/datafusion/datafusion/datafusion/proto/src/physical_plan/mod.rs:713

     Summary semver requires new minor version: 0 major and 1 minor checks failed
    Finished [ 122.639s] datafusion-proto

[codecov-commenter]

Codecov Report

❌ Patch coverage is 73.05195% with 83 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.65%. Comparing base (16471ee) to head (6794927).

Files with missing lines	Patch %	Lines
datafusion/physical-plan/src/windows/proto.rs	75.13%	17 Missing and 29 partials ⚠️
datafusion/proto/src/physical_plan/mod.rs	4.76%	20 Missing ⚠️
...usion/physical-plan/src/windows/window_agg_exec.rs	84.05%	6 Missing and 5 partials ⚠️
...ysical-plan/src/windows/bounded_window_agg_exec.rs	81.81%	3 Missing and 3 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #23780      +/-   ##
==========================================
- Coverage   80.69%   80.65%   -0.05%     
==========================================
  Files        1090     1091       +1     
  Lines      370357   370564     +207     
  Branches   370357   370564     +207     
==========================================
+ Hits       298859   298860       +1     
- Misses      53684    53870     +186     
- Partials    17814    17834      +20     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[adriangb]

Thanks!

[adriangb]

Could we put these under a mod proto here or as a file alongside this one?

[Phoenix500526]

Done

[adriangb]

One request: consider keeping the explanatory comments from #23496. The port strips them, and
a few are rationale rather than narration:

• The note on input_order_mode: None in WindowAggExec::try_to_proto — that both executors
  share the single PhysicalPlanType::Window variant and are told apart only by the presence of
  input_order_mode. This is the non-obvious invariant of the whole file; someone "tidying up"
  that field later would silently break BoundedWindowAggExec decoding.
• The mirroring note on the encode side of BoundedWindowAggExec, that Some(input_order_mode)
  is what routes the shared decode arm back to the bounded exec.
• The WindowUDFExpr::args() comment explaining that it returns the full, unfiltered argument
  list so every argument survives the round-trip (as opposed to window_expr.expressions()).
• The // TODO: Remove extended_schema if functions are all UDAF marker that was carried in the
  original datafusion-proto code — that one is pre-existing and shouldn't be dropped as part of
  a move.

[Phoenix500526]

One request: consider keeping the explanatory comments from #23496. The port strips them, and a few are rationale rather than narration:

• The note on input_order_mode: None in WindowAggExec::try_to_proto — that both executors
  share the single PhysicalPlanType::Window variant and are told apart only by the presence of
  input_order_mode. This is the non-obvious invariant of the whole file; someone "tidying up"
  that field later would silently break BoundedWindowAggExec decoding.
• The mirroring note on the encode side of BoundedWindowAggExec, that Some(input_order_mode)
  is what routes the shared decode arm back to the bounded exec.
• The WindowUDFExpr::args() comment explaining that it returns the full, unfiltered argument
  list so every argument survives the round-trip (as opposed to window_expr.expressions()).
• The // TODO: Remove extended_schema if functions are all UDAF marker that was carried in the
  original datafusion-proto code — that one is pre-existing and shouldn't be dropped as part of
  a move.

Done