[
https://issues.apache.org/jira/browse/FLINK-21564?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17484792#comment-17484792
]
Pedro Silva commented on FLINK-21564:
-------------------------------------
Hi, [~ym], thanks for the info. The PR is accepted but I don't know why it
wasn't merged yet. I'll ping the reviewer of PR
#[15344|[https://github.com/apache/flink/pull/15344]] to try to understand why
it wasn't merged. Thank you!
> CommonTestUtils.waitUntilCondition could fail with condition meets before
> -------------------------------------------------------------------------
>
> Key: FLINK-21564
> URL: https://issues.apache.org/jira/browse/FLINK-21564
> Project: Flink
> Issue Type: Bug
> Components: Tests
> Reporter: Kezhu Wang
> Assignee: Pedro Silva
> Priority: Minor
> Labels: auto-unassigned, pull-request-available
> Fix For: 1.15.0
>
>
> {code}
> public static void waitUntilCondition(
> SupplierWithException<Boolean, Exception> condition,
> Deadline timeout,
> long retryIntervalMillis,
> String errorMsg)
> throws Exception {
> while (timeout.hasTimeLeft() && !condition.get()) {
> final long timeLeft = Math.max(0, timeout.timeLeft().toMillis());
> Thread.sleep(Math.min(retryIntervalMillis, timeLeft));
> }
> if (!timeout.hasTimeLeft()) {
> throw new TimeoutException(errorMsg);
> }
> }
> {code}
> The timeout could run off between truth condition and last checking.
> Besides this, I also see time-out blocking condition in some tests, the
> combination could be worse.
> Not a big issue, but worth to be aware of and solved.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
