-
Notifications
You must be signed in to change notification settings - Fork 13.8k
[FLINK-38364][streaming-java] Implement async state version of ProcessingTimeoutTrigger #27027
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
e4cd5ed
to
27c98b3
Compare
@Zakelly Could you have a look at this pr, which resolves this ticket. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR! I was thinking it's better we use this converter in AsyncTriggerConverter
27c98b3
to
5903d2d
Compare
@Zakelly Thank you for review. That's a good idea. I have intergrated The changes in brief:
|
5903d2d
to
14a408b
Compare
…singTimeoutTrigger
14a408b
to
5d6ddc9
Compare
@flinkbot run azure |
Implement
AsyncProcessingTimeoutTrigger
, the async state version ofProcessingTimeoutTrigger
Before this pr,
ProcessingTimeoutTrigger
only supports sync state mode. After this pr, user can useAsyncProcessingTimeoutTrigger
, and enable async state mode.Brief change log
AsyncProcessingTimeoutTrigger
, supportProcessingTimeoutTrigger
in async mode.Verifying this change
ProcessingTimeoutTriggerTest
to test both sync and async state mode.Does this pull request potentially affect one of the following parts:
@Public(Evolving)
: no