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

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

                Author: ASF GitHub Bot
            Created on: 08/Jul/20 14:19
            Start Date: 08/Jul/20 14:19
    Worklog Time Spent: 10m 
      Work Description: arjun4084346 commented on a change in pull request 
#3056:
URL: https://github.com/apache/incubator-gobblin/pull/3056#discussion_r451581777



##########
File path: 
gobblin-cluster/src/main/java/org/apache/gobblin/cluster/GobblinClusterUtils.java
##########
@@ -112,6 +116,26 @@ public static Path getJobStateFilePath(boolean 
usingStateStore, Path appWorkPath
     return jobStateFilePath;
   }
 
+  /**
+   * Return the system properties from the input {@link Config} instance

Review comment:
       1) The method is not returning anything.
   2) Please describe the special treatment of 
java.io.tmpdir/TMP_DIR.YARN_CACHE config




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


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

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

> Provide an option to configure the java tmp dir to the Yarn cache location  
> ----------------------------------------------------------------------------
>
>                 Key: GOBBLIN-1209
>                 URL: https://issues.apache.org/jira/browse/GOBBLIN-1209
>             Project: Apache Gobblin
>          Issue Type: Improvement
>          Components: gobblin-yarn
>    Affects Versions: 0.15.0
>            Reporter: Sudarshan Vasudevan
>            Assignee: Abhishek Tiwari
>            Priority: Major
>             Fix For: 0.15.0
>
>          Time Spent: 50m
>  Remaining Estimate: 0h
>
> This PR allows the tmp dir to be set to the container's cache location 
> overriding the default setting of /tmp. This requires the the tmp dir system 
> property to dynamically configured to the cache location.  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to