[kudu-CR] subprocess: allow Call() to read both stdout and stderr

2016-08-24 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4057 to look at the new patch set (#2). Change subject: subprocess: allow Call() to read both stdout and stderr

[kudu-CR] subprocess: allow Call() to read both stdout and stderr

2016-08-23 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: subprocess: allow Call() to read both stdout and stderr .. Patch Set 1: Code-Review+1 Test passes on OS X -- To view, visit http://gerrit.cloudera.org:8080/4057

[kudu-CR] subprocess: allow Call() to read both stdout and stderr

2016-08-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: subprocess: allow Call() to read both stdout and stderr .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4057/1/src/kudu/util/subprocess.cc File

[kudu-CR] subprocess: allow Call() to read both stdout and stderr

2016-08-23 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: subprocess: allow Call() to read both stdout and stderr .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/4057/1/src/kudu/util/subprocess.cc File

[kudu-CR] subprocess: allow Call() to read both stdout and stderr

2016-08-18 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: subprocess: allow Call() to read both stdout and stderr .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2986/ -- To view, visit

[kudu-CR] subprocess: allow Call() to read both stdout and stderr

2016-08-18 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4057 to review the following change. Change subject: subprocess: allow Call() to read both stdout and stderr ..