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

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

                Author: ASF GitHub Bot
            Created on: 14/Feb/25 12:01
            Start Date: 14/Feb/25 12:01
    Worklog Time Spent: 10m 
      Work Description: abhishekmjain commented on code in PR #4097:
URL: https://github.com/apache/gobblin/pull/4097#discussion_r1956038447


##########
gobblin-temporal/src/main/java/org/apache/gobblin/temporal/joblauncher/GobblinJobLauncher.java:
##########
@@ -137,13 +137,9 @@ public GobblinJobLauncher(Properties jobProps, Path 
appWorkDir,
   @Override
   public void close() throws IOException {
     try {
-      executeCancellation();

Review Comment:
   Can you elaborate on what documentation you feel should be added?
   The `executeCancellation` is a No-Op function implemented in 
`GobblinJobLauncher`





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

    Worklog Id:     (was: 957117)
    Time Spent: 2h 40m  (was: 2.5h)

> Fix Temporal metrics scope closure bug
> --------------------------------------
>
>                 Key: GOBBLIN-2194
>                 URL: https://issues.apache.org/jira/browse/GOBBLIN-2194
>             Project: Apache Gobblin
>          Issue Type: Bug
>          Components: gobblin-metrics
>            Reporter: Abhishek Jain
>            Assignee: Issac Buenrostro
>            Priority: Major
>          Time Spent: 2h 40m
>  Remaining Estimate: 0h
>
> As part of https://issues.apache.org/jira/browse/GOBBLIN-2192 a bug was 
> introduced where the metrics scope object was not closed resulting in AM 
> container not shutting down.
> Metrics scope should be closed wherever initialized.



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

Reply via email to