fix(python): Replace _kwargs in collect method

[jackxxu]

when I was incorporating @ritchie46's suggestion in disabling CSE in the collect method in #19480, I initially used

ldf.collect(common_subplan_elim=False)

it didn't complain but produces no performance improvements that I was expecting, until I looked it up and found the keyword to be comm_subplan_elim instead.

this PR solves this potential pitfull for developers by replacing the _kwargs with spelled out keyword arguments, to let python do the checking for us and raise TypeError if not correct.

typically, unless a method either forwards/delegates the kwargs to another method or it takes whatever, it is reasonable to assume that the method spells out its interface.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.91%. Comparing base (0b0a914) to head (9272ff6).
Report is 46 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #19618      +/-   ##
==========================================
- Coverage   79.96%   79.91%   -0.05%     
==========================================
  Files        1536     1536              
  Lines      211374   211663     +289     
  Branches     2445     2447       +2     
==========================================
+ Hits       169034   169160     +126     
- Misses      41785    41948     +163     
  Partials      555      555              

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

[cmdlineluser]

This will close #16525

[ritchie46]

post_opt_callback is not intended for public use. It is private. We should instead check the content of _kwargs for items that aren't allowed (if any).

[jackxxu]

@ritchie46, commits added