----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48240/#review136154 -----------------------------------------------------------
Patch looks great! Reviews applied: [48240] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker_build.sh - Mesos ReviewBot On June 4, 2016, 2:25 a.m., Silas Snider wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48240/ > ----------------------------------------------------------- > > (Updated June 4, 2016, 2:25 a.m.) > > > Review request for mesos and Jiang Yan Xu. > > > Bugs: MESOS-5543 > https://issues.apache.org/jira/browse/MESOS-5543 > > > Repository: mesos > > > Description > ------- > > This is necessary to enable bash subshell redirection within the > container. > > > Diffs > ----- > > src/linux/fs.cpp 3190fcec572eddef3e3d5e81f5e508798deee1bd > > Diff: https://reviews.apache.org/r/48240/diff/ > > > Testing > ------- > > I ran a task using the filesystem/linux isolator, that used bash subshell > redirection (i.e., `grep -q -E 'something' <(tail -q -c +0 -f somefile)`. > Without this line, the task fails. With it, it runs just fine. > > > Thanks, > > Silas Snider > >