You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Ayush Saxena (Jira)" <ji...@apache.org> on 2020/01/29 19:06:00 UTC
[jira] [Commented] (FLINK-15740) Remove Deadline#timeLeft()
[ https://issues.apache.org/jira/browse/FLINK-15740?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17026126#comment-17026126 ]
Ayush Saxena commented on FLINK-15740:
--------------------------------------
Thanx [~chesnay] for the putting this up.
Have raised PR. [https://github.com/apache/flink/pull/10966]
Please help review!!!
> Remove Deadline#timeLeft()
> --------------------------
>
> Key: FLINK-15740
> URL: https://issues.apache.org/jira/browse/FLINK-15740
> Project: Flink
> Issue Type: Improvement
> Components: API / DataStream, Tests
> Reporter: Chesnay Schepler
> Priority: Major
> Labels: pull-request-available
> Fix For: 1.11.0
>
> Time Spent: 10m
> Remaining Estimate: 0h
>
> As shown in FLINK-13662, {{Deadline#timeLeft()}} is conceptually broken since there is no reliable way to call said method while ensuring that
> a) the value is non-negative (desired since most time-based APIs reject negative values)
> b) the value sign (+,-) corresponds to preceding calls to {{#hasTimeLeft()}}
>
> As a result any usage of the following form is unreliable and obfuscating error messages.
> {code:java}
> while (deadline.hasTimeLeft()) {
> doSomething(deadline.timeLeft());
> } {code}
>
> All existing usage should be migrate to either
> {code:java}
> while (deadline.hasTimeLeft()) {
> doSomething();
> } {code}
> or
> {code:java}
> while (true) {
> doSomething(deadline.timeLeftIfAny());
> }
> {code}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)