[FLINK-40095][table-planner] Fix JSON aggregation functions failing in group window aggregate

[xuyangzhong]

What is the purpose of the change

This pull request fixes JSON aggregation functions (JSON_OBJECTAGG / JSON_ARRAYAGG) failing when used in a group window aggregate (e.g. GROUP BY TUMBLE(...)). Previously, WrapJsonAggFunctionArgumentsRule only matched LogicalAggregate, so JSON aggregate arguments were never wrapped into JSON_STRING when the aggregation was a LogicalWindowAggregate, leading to a failure during planning. This change extends the rule to also cover window aggregates.

Brief change log

• Generalized WrapJsonAggFunctionArgumentsRule to operate on the base Aggregate class and split the single INSTANCE into AGGREGATE_INSTANCE (for LogicalAggregate) and WINDOW_AGGREGATE_INSTANCE (for LogicalWindowAggregate), with a generic onAggregateClass(Class) config factory
• Added a hints field to WindowAggregate and its subclasses (LogicalWindowAggregate, FlinkLogicalWindowAggregate), and implemented withHints on LogicalWindowAggregate so the rule's marker hint (used to prevent repeated transformation) is preserved
• Registered both rule instances in FlinkBatchRuleSets and FlinkStreamRuleSets, and updated the HINT_NAME_JSON_AGGREGATE_WRAPPED excluded-rules strategy in FlinkHintStrategies

Verifying this change

New tests are added to verify this pr.

Does this pull request potentially affect one of the following parts:

• Dependencies (does it add or upgrade a dependency): no
• The public API, i.e., is any changed class annotated with @Public(Evolving): no
• The serializers: no
• The runtime per-record code paths (performance sensitive): no
• Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
• The S3 file system connector: no

Documentation

• Does this pull request introduce a new feature? no
• If yes, how is the feature documented?

[flinkbot]

CI report:

• d7235a7 Azure: SUCCESS

Bot commands

The @flinkbot bot supports the following commands:

• @flinkbot run azure re-run the last Azure build

[raminqaf]

Thanks @xuyangzhong! I did a quick first pass.

[raminqaf]

Here and everywhere else please use List.of()

[raminqaf]

Beside that, shouldn't we here also pass the hint?

[xuyangzhong]

This is because FlinkLogicalWindowTableAggregate is not a hintable node.

[xuyangzhong]

Thanks for your review @raminqaf ! Updated

[raminqaf]

Thanks for addressing! Left two nits. Should be good in the next round!

[raminqaf]

nit:

Suggested change

	util.List.of(),
	List.of(),

[xuyangzhong]

When I imported java.util.List and changed it to List.of, IntelliJ IDEA threw the following warning, asking me to change it back to util.List.of. What about just keeping util.List.of, WDYT?

[raminqaf]

Thanks for verifying this! It is okay like this.

[xuyangzhong]

Hi, @raminqaf could you please take another look again? Please let me know if you have any other concerns.

[raminqaf]

@xuyangzhong Thanks for applying the feedback so fast. LGTM!