Copilot commented on code in PR #28827:
URL: https://github.com/apache/flink/pull/28827#discussion_r3735193609
##########
flink-table/flink-table-api-java/src/main/java/org/apache/flink/table/api/config/EarlyFireJoinHintOptions.java:
##########
@@ -36,6 +36,18 @@
@PublicEvolving
public class EarlyFireJoinHintOptions {
+ /** The only operator kind the EARLY_FIRE hint applies to. */
Review Comment:
The comment implies the EARLY_FIRE hint can *only* ever apply to interval
joins, but this PR introduces `target` specifically as a forward-compatibility
guard for potentially supporting additional operator kinds later. Wording this
as "currently supported" avoids locking in an incorrect API contract in a
`@PublicEvolving` class.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
