[
https://issues.apache.org/jira/browse/GOBBLIN-2032?focusedWorklogId=913506&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-913506
]
ASF GitHub Bot logged work on GOBBLIN-2032:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 08/Apr/24 19:03
Start Date: 08/Apr/24 19:03
Worklog Time Spent: 10m
Work Description: phet commented on code in PR #3911:
URL: https://github.com/apache/gobblin/pull/3911#discussion_r1556331040
##########
gobblin-yarn/src/main/java/org/apache/gobblin/yarn/GobblinYarnAppLauncher.java:
##########
@@ -204,7 +204,7 @@ public class GobblinYarnAppLauncher {
private final Optional<String> appMasterJvmArgs;
- private final Path sinkLogRootDir;
+ private final String sinkLogRootDir;
Review Comment:
does anything need to change where it's used, now that this is a `String`
and no longer a `Path`?
Issue Time Tracking
-------------------
Worklog Id: (was: 913506)
Time Spent: 1h (was: 50m)
> Support making log dir sink optional
> ------------------------------------
>
> Key: GOBBLIN-2032
> URL: https://issues.apache.org/jira/browse/GOBBLIN-2032
> Project: Apache Gobblin
> Issue Type: Improvement
> Components: gobblin-cluster
> Reporter: William Lo
> Assignee: Hung Tran
> Priority: Major
> Time Spent: 1h
> Remaining Estimate: 0h
>
> In Gobblin cluster, there is a configuration to set the log sink in order to
> create a log copier service which copies logs to HDFS from Yarn cluster.
> Since Yarn has a built in log aggregator, we should have a way to disable
> this log sink since it becomes redundant, so we should look into ways to make
> the log directory optional.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
