Make udaf_default_* display/schema name helpers non-generic (214 KB smaller binary)

[alamb]

Which issue does this PR close?

• Part of [EPIC] Reduce binary size #24727

Rationale for this change

The default udaf_default_* display / schema name helpers in datafusion/expr/src/udaf.rs are instantiated for each UDAF

What changes are included in this PR?

1. Move the default implementations of the five display / schema name methods into non-generic builder structs so each body is compiled only once
2. Update the AggregateUDFImpl default methods to use the builders
3. Mark existing functions as deprecated
4. Add a doc example

Code size

Measured per the methodology in #24727, at merge base d66f4b9 and on this branch (39d0e53).

cargo llvm-lines --release -p datafusion-functions-aggregate --lib (29 AggregateUDFImpl implementations):

	main (d66f4b9)	this branch	change
udaf_default_* display/schema name helpers	212,048 IR lines (2,755 functions)	0	-212,048 (-100%)
Crate total	2,112,337 IR lines (39,704 copies)	1,900,949 IR lines (36,936 copies)	-211,388 (-10.0%)

cargo llvm-lines --release -p datafusion-expr --lib (the builder bodies now compile here exactly once):

	main (d66f4b9)	this branch	change
Crate total	1,265,863 IR lines (26,406 copies)	1,229,636 IR lines (25,925 copies)	-36,227 (-2.9%)

cargo build --release --bin datafusion-cli:

	main (d66f4b9)	this branch (39d0e53)	change
Binary size	81,580,368 bytes	81,365,536 bytes	-214,832 bytes (-0.26%)

Are these changes tested?

Covered by existing tests

Are there any user-facing changes?

Yes, but not breaking:

[alamb]

This Diff looks lage, but if you ignore whitespace via https://github.com/apache/datafusion/pull/24728/changes?w=1 it is mich smaller (basically adding a bunch of docs and a builder)

[alamb]

The core rationale is to use a single non generic function -- I also made a struct like builder to make the code easier to see too

[alamb]

this logic is all the same -- there is just now some more builder ceremony logic

[alamb]

Here is a new builder for schema name -- it is basically the same pattern for all of the other types

[codecov-commenter]

Codecov Report

❌ Patch coverage is 79.39698% with 41 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.47%. Comparing base (c56a7ab) to head (3ef58be).
⚠️ Report is 14 commits behind head on main.

Files with missing lines	Patch %	Lines
datafusion/expr/src/udaf.rs	79.39%	21 Missing and 20 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #24728      +/-   ##
==========================================
- Coverage   81.47%   81.47%   -0.01%     
==========================================
  Files        1122     1122              
  Lines      404140   404185      +45     
  Branches   404140   404185      +45     
==========================================
+ Hits       329284   329292       +8     
- Misses      55546    55577      +31     
- Partials    19310    19316       +6     

☔ 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.

[alamb]

Extended teset failure seems to be failing on main too

    joins::hash_join::exec::tests::test_partitioned_null_equal_dynamic_filter_keeps_probe_nulls_for_build_logical_null

Here is a ticket

• Extended tests seem to be failing on main #24734

[jayzhan211]

Thanks @alamb , LGTM