Hello Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/15584 to look at the new patch set (#5). Change subject: env: add a fifo class ...................................................................... env: add a fifo class This adds a basic Fifo class and a PosixFifo variant that encapsulates the usage of mkfifo and associated functionality. My immediate need is just to return the FDs of the opened fifo, so the interface is quite bare. That said, this will be good enough to move the SubprocessServer away from using the process stdout FD for IPC. Change-Id: I23920457d695b84509937694daea53d61f445e27 --- M src/kudu/util/env-test.cc M src/kudu/util/env.h M src/kudu/util/env_posix.cc 3 files changed, 132 insertions(+), 21 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/84/15584/5 -- To view, visit http://gerrit.cloudera.org:8080/15584 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I23920457d695b84509937694daea53d61f445e27 Gerrit-Change-Number: 15584 Gerrit-PatchSet: 5 Gerrit-Owner: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241)