perf(hash agg): introduce const EOWC: bool generic argument to HashAggExecutor

[stdrc]

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.

What's changed and what's your intention?

Make Emit-On-Window-Close mode a generic argument instead of a normal argument in HashAggExecutor.

Checklist

• I have written necessary rustdoc comments.
• I have added necessary unit tests and integration tests.
• I have added test labels as necessary.
• I have added fuzzing tests or opened an issue to track them.
• My PR contains breaking changes.
• My PR changes performance-critical code, so I will run (micro) benchmarks and present the results.
• My PR contains critical fixes that are necessary to be merged into the latest release.

Documentation

• My PR needs documentation updates.

Release note

[stdrc]

• fix(hash agg): split AggGroup inter states and outputs change building, fixing old groups not deleted bug #20435
• perf(hash agg): introduce const EOWC: bool generic argument to HashAggExecutor #20434 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[xxchan]

Can you test whether this increase the compile time of cargo build -p risingwave_stream too much?

And the code size increase (diff of cargo bloat)

[stdrc]

Can you test whether this increase the compile time of cargo build -p risingwave_stream too much?

main	this branch
46.53s	53.28s

Maybe it's better to change back(