PatrickRen opened a new pull request, #20760:
URL: https://github.com/apache/flink/pull/20760

   ## What is the purpose of the change
   
   This pull request adds IdlenessManager for main and per-split output in 
source operator to mark idleness only if both output are idle
   
   
   ## Brief change log
   
   - Add IdlenessManager for main and per-split output in source operator to 
mark idleness only if both output are idle
   - Improve WatermarksWithIdleness to avoid sending idleness event repeatedly 
on idle
   - Migrate SourceOperatorEventTimeTest to JUnit 5 style
   
   
   ## Verifying this change
   
   This change added tests and can be verified as follows:
   - SourceOperatorEventTimeTest#testMainAndPerSplitWatermarkIdleness, that 
validates the reader output is marked as idle only if both main and per-split 
source are idle
   
   ## Does this pull request potentially affect one of the following parts:
   
     - Dependencies (does it add or upgrade a dependency): (yes / **no**)
     - The public API, i.e., is any changed class annotated with 
`@Public(Evolving)`: (yes / **no**)
     - The serializers: (yes / **no** / don't know)
     - The runtime per-record code paths (performance sensitive): (yes / **no** 
/ don't know)
     - Anything that affects deployment or recovery: JobManager (and its 
components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / **no** / don't 
know)
     - The S3 file system connector: (yes / **no** / don't know)
   
   ## Documentation
   
     - Does this pull request introduce a new feature? (yes / **no**)
     - If yes, how is the feature documented? (**not applicable** / docs / 
JavaDocs / not documented)
   


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to