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)

Reply via email to