Repository: hadoop Updated Branches: refs/heads/branch-2.8 9eebbcf45 -> c43ee0058
YARN-3760. FSDataOutputStream leak in AggregatedLogFormat.LogWriter.close(). Contributed by Haibo Chen. (cherry picked from commit c26ccf1adb3a72df3f68e1150b86b813c691203a) (cherry picked from commit 9d9087a67ad599e678948d5c1036e51cb8cf5e3a) (cherry picked from commit d77cada182ef805679dc79498d6b6cc6e7f6b6d4) Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/c43ee005 Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/c43ee005 Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/c43ee005 Branch: refs/heads/branch-2.8 Commit: c43ee00587e939ab5fdcfb9e717eb5948d12e6f8 Parents: 9eebbcf Author: Junping Du <junping...@apache.org> Authored: Wed Apr 12 09:34:34 2017 -0700 Committer: Junping Du <junping...@apache.org> Committed: Wed Apr 12 14:17:04 2017 -0700 ---------------------------------------------------------------------- .../yarn/logaggregation/AggregatedLogFormat.java | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hadoop/blob/c43ee005/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/logaggregation/AggregatedLogFormat.java ---------------------------------------------------------------------- diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/logaggregation/AggregatedLogFormat.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/logaggregation/AggregatedLogFormat.java index 5e4648a..38c1e6a 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/logaggregation/AggregatedLogFormat.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/logaggregation/AggregatedLogFormat.java @@ -473,14 +473,15 @@ public class AggregatedLogFormat { @Override public void close() { - if (writer != null) { - try { - this.writer.close(); - } catch (IOException e) { - LOG.warn("Exception closing writer", e); + try { + if (writer != null) { + writer.close(); } + } catch (Exception e) { + LOG.warn("Exception closing writer", e); + } finally { + IOUtils.closeStream(this.fsDataOStream); } - IOUtils.closeStream(fsDataOStream); } } --------------------------------------------------------------------- To unsubscribe, e-mail: common-commits-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-commits-h...@hadoop.apache.org
