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

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

                Author: ASF GitHub Bot
            Created on: 12/Mar/24 03:36
            Start Date: 12/Mar/24 03:36
    Worklog Time Spent: 10m 
      Work Description: phet commented on code in PR #3892:
URL: https://github.com/apache/gobblin/pull/3892#discussion_r1520801829


##########
gobblin-service/src/main/java/org/apache/gobblin/service/modules/scheduler/GobblinServiceJobScheduler.java:
##########
@@ -120,7 +122,7 @@ public class GobblinServiceJobScheduler extends 
JobScheduler implements SpecCata
   protected final Optional<UserQuotaManager> quotaManager;
   protected final Optional<FlowTriggerHandler> flowTriggerHandler;
   @Getter
-  protected final Map<String, Spec> scheduledFlowSpecs;
+  protected final Map<String, FlowSpec> scheduledFlowSpecs;

Review Comment:
   NBD, this may be for the best... but I can't seem to figure out: where is 
the initial motivation coming in?  e.g. I was looking for a changed method 
signature, but didn't notice one.  is it entirely a preference to change the 
type of these various members?
   
   (again, I'm not calling into question the decision to change, more wanting 
to understand motivation for the decision.)





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

    Worklog Id:     (was: 909330)
    Time Spent: 50m  (was: 40m)

> create instances of new new class created in PR #3858
> -----------------------------------------------------
>
>                 Key: GOBBLIN-2013
>                 URL: https://issues.apache.org/jira/browse/GOBBLIN-2013
>             Project: Apache Gobblin
>          Issue Type: Task
>            Reporter: Arjun Singh Bora
>            Priority: Major
>          Time Spent: 50m
>  Remaining Estimate: 0h
>




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

Reply via email to