Add AftersynchronizedProcessing Time as continuation trigger#35913
Add AftersynchronizedProcessing Time as continuation trigger#35913kennknowles merged 6 commits intoapache:masterfrom
Conversation
|
Tested YAML Iceberg pipeline working as expected after introducing this trigger . Failed tests look irrelevant. |
|
Stopping reviewer notifications for this pull request: review requested by someone other than the bot, ceding control. If you'd like to restart, comment |
|
Hi @tarun-google - please fix the lint issues. Looks like some imports are incorrectly sorted. Thanks! |
ahmedabu98
left a comment
There was a problem hiding this comment.
Generally LGTM, but I think @kennknowles should do a pass too
kennknowles
left a comment
There was a problem hiding this comment.
Looks good. I did a quick pass to check against Java just to make sure they are consistent. (not worrying too much about if they are ideal, just as long as they match it is the right thing to do )
|
Run Python_Transforms PreCommit 3.12 |
|
Please get the tests green (either by re-running or fixing real issues) and I'll merge. |
|
If there are new very-flaky tests, please file bugs for them. There may already be bugs filed based on other workflows being flaky, but figuring out which tests are the flakes would be very useful. |
|
Run Python_Examples PreCommit 3.10 |
|
Run Python_Integration PreCommit 3.9 |
|
Thanks @kennknowles & @derrickaw for triggering 👍 I will follow up the tests |
|
Run Yaml_Xlang_Direct PreCommit |
|
Run Python_Runners PreCommit 3.9 |
|
All green Yay 🥇 |
|
Runner V2 does not support this yet. Revert this. |
Fixes #34212
In extension to previously discarded work : #14060
GitHub Actions Tests Status (on master branch)
See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.