Github user chandnisingh commented on a diff in the pull request:
https://github.com/apache/incubator-apex-malhar/pull/268#discussion_r63591292
--- Diff:
library/src/main/java/com/datatorrent/lib/io/fs/FileSplitterInput.java ---
@@ -515,6 +521,9 @@ protected ScannedFileInfo createScannedFileInfo(Path
parentPath, FileStatus pare
protected static boolean skipFile(@SuppressWarnings("unused") @NotNull
Path path, @NotNull Long modificationTime,
Long lastModificationTime) throws IOException
{
+ if(lastModificationTime == null) {
--- End diff --
The check lastModificationTime == null becomes redundant in the next line.
Why was this change necessary?
Please revert it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
