[ 
https://issues.apache.org/jira/browse/GOBBLIN-1708?focusedWorklogId=809661&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-809661
 ]

ASF GitHub Bot logged work on GOBBLIN-1708:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 16/Sep/22 21:03
            Start Date: 16/Sep/22 21:03
    Worklog Time Spent: 10m 
      Work Description: arjun4084346 commented on code in PR #3563:
URL: https://github.com/apache/gobblin/pull/3563#discussion_r973390481


##########
gobblin-data-management/src/main/java/org/apache/gobblin/data/management/copy/TimeAwareRecursiveCopyableDataset.java:
##########
@@ -134,9 +134,40 @@ protected List<FileStatus> getFilesAtPath(FileSystem fs, 
Path path, PathFilter f
     return recursivelyGetFilesAtDatePath(fs, path, "", fileFilter, 1, 
startDate, endDate, formatter);
   }
 
+  public Boolean checkPathDateTimeValidity(LocalDateTime startDate, 
LocalDateTime endDate, String traversedDatePath) {

Review Comment:
   What exactly is this method doing? Please add the javadoc.
   Is it comparing two Dates? If yes, we should do it with available library 
methods.





Issue Time Tracking
-------------------

    Worklog Id:     (was: 809661)
    Time Spent: 1h 20m  (was: 1h 10m)

> Improve TimeAwareRecursiveCopyableDataset to lookback only into datefolders 
> that match range
> --------------------------------------------------------------------------------------------
>
>                 Key: GOBBLIN-1708
>                 URL: https://issues.apache.org/jira/browse/GOBBLIN-1708
>             Project: Apache Gobblin
>          Issue Type: Improvement
>            Reporter: Andy Jiang
>            Priority: Major
>          Time Spent: 1h 20m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to