[ https://issues.apache.org/jira/browse/FLINK-4496?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15510268#comment-15510268 ]
ASF GitHub Bot commented on FLINK-4496: --------------------------------------- Github user aljoscha commented on a diff in the pull request: https://github.com/apache/flink/pull/2434#discussion_r79854449 --- Diff: flink-streaming-java/src/main/java/org/apache/flink/streaming/runtime/tasks/StreamTask.java --- @@ -486,6 +472,9 @@ private void disposeAllOperators() { protected void finalize() throws Throwable { super.finalize(); if (timerService != null) { + if (!timerService.isTerminated()) { --- End diff -- Should the actual call to `shutdownService()` also be inside the if block? > Refactor the TimeServiceProvider to take a Trigerable instead of a Runnable. > ---------------------------------------------------------------------------- > > Key: FLINK-4496 > URL: https://issues.apache.org/jira/browse/FLINK-4496 > Project: Flink > Issue Type: Sub-task > Reporter: Kostas Kloudas > Assignee: Kostas Kloudas > -- This message was sent by Atlassian JIRA (v6.3.4#6332)