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

ASF GitHub Bot commented on APEXMALHAR-2078:
--------------------------------------------

Github user chandnisingh commented on a diff in the pull request:

    
https://github.com/apache/incubator-apex-malhar/pull/268#discussion_r63450096
  
    --- Diff: 
library/src/test/java/com/datatorrent/lib/io/fs/FileSplitterInputTest.java ---
    @@ -107,7 +108,7 @@ protected void starting(org.junit.runner.Description 
description)
           fileSplitterInput.setScanner(scanner);
           fileSplitterInput.getScanner().setScanIntervalMillis(500);
           fileSplitterInput.getScanner().setFilePatternRegularExp(".*[.]txt");
    -      fileSplitterInput.getScanner().setFiles(dataDirectory);
    +      fileSplitterInput.getScanner().setFiles(new 
File(dataDirectory).getAbsolutePath());
    --- End diff --
    
    @DT-Priyanka 
    The user can add relative or absolute path. We shouldn't be imposing adding 
absolute path on the user.


> Potential thread issue in FileSplitterInput class
> -------------------------------------------------
>
>                 Key: APEXMALHAR-2078
>                 URL: https://issues.apache.org/jira/browse/APEXMALHAR-2078
>             Project: Apache Apex Malhar
>          Issue Type: Bug
>            Reporter: Priyanka Gugale
>            Assignee: Priyanka Gugale
>             Fix For: 3.4.0
>
>
> The "inputDirTolastModifiedTimes" field is modified by multiple threads and 
> check pointed by operator thread. This could cause threading issues.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to