----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24700/#review50600 -----------------------------------------------------------
Did you test this manually? src/logging/logging.cpp <https://reviews.apache.org/r/24700/#comment88470> Since we're in a signal handler, we have to avoid using system calls that are not async signal safe: http://man7.org/linux/man-pages/man7/signal.7.html os::user uses sysconf and getpwuid_r, so I think the best we can do here is just print the si_uid. - Ben Mahler On Aug. 14, 2014, 3:12 p.m., Alexandra Sava wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24700/ > ----------------------------------------------------------- > > (Updated Aug. 14, 2014, 3:12 p.m.) > > > Review request for mesos and Ben Mahler. > > > Bugs: MESOS-1567 > https://issues.apache.org/jira/browse/MESOS-1567 > > > Repository: mesos-git > > > Description > ------- > > Log the user id when receiving a SIGTERM. This will make debugging more easy. > > > Diffs > ----- > > src/logging/logging.cpp b05a7e0c3cb5fcdf0ffd8ee7e1bb970bf71622ea > > Diff: https://reviews.apache.org/r/24700/diff/ > > > Testing > ------- > > > Thanks, > > Alexandra Sava > >
