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

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

                Author: ASF GitHub Bot
            Created on: 25/Apr/24 05:30
            Start Date: 25/Apr/24 05:30
    Worklog Time Spent: 10m 
      Work Description: phet commented on code in PR #3931:
URL: https://github.com/apache/gobblin/pull/3931#discussion_r1578888790


##########
gobblin-yarn/src/main/java/org/apache/gobblin/yarn/GobblinYarnAppLauncher.java:
##########
@@ -884,7 +884,8 @@ private AbstractTokenRefresher buildTokenRefreshManager() 
throws IOException {
 
   @VisibleForTesting
   void cleanUpAppWorkDirectory(ApplicationId applicationId) throws IOException 
{
-    FileSystem fs = 
GobblinClusterUtils.buildNewInstanceFileSystem(this.config, 
this.yarnConfiguration);
+    // Create a new filesystem as this.fs may have been closed by the Yarn 
Application, and FS.get() will return a cached instance of the closed FS

Review Comment:
   bingo!  glad to preserve this hard-won understanding





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

    Worklog Id:     (was: 916316)
    Time Spent: 2h 50m  (was: 2h 40m)

> Allow configurable token file paths and cluster work directories to allow for 
> easy cleanup after yarn app closes
> ----------------------------------------------------------------------------------------------------------------
>
>                 Key: GOBBLIN-2050
>                 URL: https://issues.apache.org/jira/browse/GOBBLIN-2050
>             Project: Apache Gobblin
>          Issue Type: Improvement
>            Reporter: William Lo
>            Priority: Major
>          Time Spent: 2h 50m
>  Remaining Estimate: 0h
>
> Gobblin Yarn Application Launcher has some issues where directories used for 
> the job can persist after the job ends. It also creates a number of temporary 
> files which can grow out of control.
> We want to be able to:
> 1. Clean up directories effectively
> 2. Use explicit paths to allow for consolidation of temp files to be under 
> the same folder for token renewal and cluster work files.



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

Reply via email to