Repository: hadoop Updated Branches: refs/heads/branch-3.1 d5d987550 -> 7cbb9597c
YARN-8515. container-executor can crash with SIGPIPE after nodemanager restart. Contributed by Jim Brennan (cherry picked from commit 17118f446c2387aa796849da8b69a845d9d307d3) Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/7cbb9597 Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/7cbb9597 Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/7cbb9597 Branch: refs/heads/branch-3.1 Commit: 7cbb9597c43d0e4270a64d28b5521941ce940a1a Parents: d5d9875 Author: Jason Lowe <jl...@apache.org> Authored: Fri Jul 13 10:05:25 2018 -0500 Committer: Jason Lowe <jl...@apache.org> Committed: Fri Jul 13 10:06:38 2018 -0500 ---------------------------------------------------------------------- .../src/main/native/container-executor/impl/main.c | 6 ++++++ 1 file changed, 6 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hadoop/blob/7cbb9597/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/native/container-executor/impl/main.c ---------------------------------------------------------------------- diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/native/container-executor/impl/main.c b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/native/container-executor/impl/main.c index c54fd3e..3d7b19a 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/native/container-executor/impl/main.c +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/native/container-executor/impl/main.c @@ -31,6 +31,7 @@ #include <unistd.h> #include <stdlib.h> #include <string.h> +#include <signal.h> static void display_usage(FILE *stream) { fprintf(stream, @@ -112,6 +113,11 @@ static void open_log_files() { if (ERRORFILE == NULL) { ERRORFILE = stderr; } + + // There may be a process reading from stdout/stderr, and if it + // exits, we will crash on a SIGPIPE when we try to write to them. + // By ignoring SIGPIPE, we can handle the EPIPE instead of crashing. + signal(SIGPIPE, SIG_IGN); } /* Flushes and closes log files */ --------------------------------------------------------------------- To unsubscribe, e-mail: common-commits-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-commits-h...@hadoop.apache.org
