Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15584 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/15584 Reviewed-by: Adar Dembo <[email protected]> Reviewed-by: Hao Hao <[email protected]> Tested-by: Kudu Jenkins --- 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(-) Approvals: Adar Dembo: Looks good to me, approved Hao Hao: Looks good to me, approved Kudu Jenkins: Verified -- 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: merged Gerrit-Change-Id: I23920457d695b84509937694daea53d61f445e27 Gerrit-Change-Number: 15584 Gerrit-PatchSet: 6 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241)
