[GitHub] [flink] XComp commented on pull request #22572: [FLINK-29618] Remove Timeout annotation in YARNSessionFIFOSecuredITCase

2023-05-12 Thread via GitHub
XComp commented on PR #22572: URL: https://github.com/apache/flink/pull/22572#issuecomment-1545304968 Based on what observation do you conclude that CI is broken? Because [no CI build](https://dev.azure.com/apache-flink/apache-flink/_build?definitionId=2) was picked up since yesterday?

[GitHub] [flink] XComp commented on pull request #22572: [FLINK-29618] Remove Timeout annotation in YARNSessionFIFOSecuredITCase

2023-05-11 Thread via GitHub
XComp commented on PR #22572: URL: https://github.com/apache/flink/pull/22572#issuecomment-1545210878 I guess, we could, at least, remove the `@Timeout` from the related `YarnSessionFIFOITCase` as well. WDYT? And what about touching the other tests in `flink-yarn-tests`? Could you check