[kudu-CR] env: add a fifo class

2020-03-30 Thread Andrew Wong (Code Review)
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 encapsu

[kudu-CR] env: add a fifo class

2020-03-30 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15584 ) Change subject: env: add a fifo class .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/15584 To unsubscribe,

[kudu-CR] env: add a fifo class

2020-03-30 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15584 ) Change subject: env: add a fifo class .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/15584/3/src/kudu/util/env_posix.c

[kudu-CR] env: add a fifo class

2020-03-30 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15584 ) Change subject: env: add a fifo class .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/15584 To unsubscri

[kudu-CR] env: add a fifo class

2020-03-30 Thread Andrew Wong (Code Review)
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 c

[kudu-CR] env: add a fifo class

2020-03-30 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15584 ) Change subject: env: add a fifo class .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/15584/3/src/kudu/util/env_posix.cc

[kudu-CR] env: add a fifo class

2020-03-29 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15584 ) Change subject: env: add a fifo class .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/15584/3/src/kudu/util/env_posix.c

[kudu-CR] env: add a fifo class

2020-03-29 Thread Andrew Wong (Code Review)
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 (#4). Change subject: env: add a fifo class .. env: add a fifo c

[kudu-CR] env: add a fifo class

2020-03-29 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15584 ) Change subject: env: add a fifo class .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/15584/3/src/kudu/util/env_posix.cc

[kudu-CR] env: add a fifo class

2020-03-28 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15584 to look at the new patch set (#3). Change subject: env: add a fifo class .. env: add a fifo class This a

[kudu-CR] env: add a fifo class

2020-03-28 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15584 to look at the new patch set (#2). Change subject: env: add a fifo class .. env: add a fifo class This a

[kudu-CR] env: add a fifo class

2020-03-28 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15584 Change subject: env: add a fifo class .. env: add a fifo class This adds a basic Fifo class that encapsulates the usage of mkfifo and ass