[ 
https://issues.apache.org/jira/browse/FLINK-4496?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15446151#comment-15446151
 ] 

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_r76625434
  
    --- Diff: 
flink-streaming-java/src/test/java/org/apache/flink/streaming/util/OneInputStreamOperatorTestHarness.java
 ---
    @@ -98,7 +99,7 @@ public 
OneInputStreamOperatorTestHarness(OneInputStreamOperator<IN, OUT> operato
        public OneInputStreamOperatorTestHarness(
                        OneInputStreamOperator<IN, OUT> operator,
                        ExecutionConfig executionConfig) {
    -           this(operator, executionConfig, 
DefaultTimeServiceProvider.create(Executors.newSingleThreadScheduledExecutor()));
    +           this(operator, executionConfig, null);
    --- End diff --
    
    This can directly be `DefaultTimeServiceProvider.create(mockTask, 
Executors.newSingleThreadScheduledExecutor())` instead of the `null` check 
below.


> 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)

Reply via email to