Skip to content

[FLINK-37402][table] Emit watermarks after record in TestSourceFunction and TestValuesRuntimeFunctions#26236

Merged
twalthr merged 2 commits intoapache:masterfrom
twalthr:FLINK-37402
Mar 3, 2025
Merged

[FLINK-37402][table] Emit watermarks after record in TestSourceFunction and TestValuesRuntimeFunctions#26236
twalthr merged 2 commits intoapache:masterfrom
twalthr:FLINK-37402

Conversation

@twalthr
Copy link
Contributor

@twalthr twalthr commented Feb 28, 2025

What is the purpose of the change

Fixes the testing sources by emitting watermarks after the record itself. This is in sync with non-testing operator behavior.

Brief change log

TestSourceFunction and TestValuesRuntimeFunctions updated.

Verifying this change

This change added tests and can be verified as follows: CalcTestPrograms.CURRENT_WATERMARK

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? not applicable

@flinkbot
Copy link
Collaborator

flinkbot commented Feb 28, 2025

CI report:

Bot commands The @flinkbot bot supports the following commands:
  • @flinkbot run azure re-run the last Azure build

@twalthr twalthr merged commit d5ffb88 into apache:master Mar 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants