[kudu-CR] KUDU-2971 p1: add subprocess module

Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14425 ) Change subject: KUDU-2971 p1: add subprocess module .. KUDU-2971 p1: add subprocess module Utility classes exist that allow for IPC

[kudu-CR] KUDU-2971 p1: add subprocess module

Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14425 ) Change subject: KUDU-2971 p1: add subprocess module .. Patch Set 5: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2971 p1: add subprocess module

Hello Tidy Bot, Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14425 to look at the new patch set (#5). Change subject: KUDU-2971 p1: add subprocess module

[kudu-CR] KUDU-2971 p1: add subprocess module

Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/14425 ) Change subject: KUDU-2971 p1: add subprocess module .. Patch Set 4: (8 comments) > Patch Set 2: > > > Patch Set 2: Code-Review+1 > > > >

[kudu-CR] KUDU-2971 p1: add subprocess module

Hello Tidy Bot, Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14425 to look at the new patch set (#4). Change subject: KUDU-2971 p1: add subprocess module

[kudu-CR] KUDU-2971 p1: add subprocess module

Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/14425 ) Change subject: KUDU-2971 p1: add subprocess module .. Patch Set 3: (3 comments) I added a few comments to supbprocess_protocol.cc -

[kudu-CR] KUDU-2971 p1: add subprocess module

Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14425 ) Change subject: KUDU-2971 p1: add subprocess module .. Patch Set 2: > Patch Set 2: Code-Review+1 > > (1 comment) > > I think this

[kudu-CR] KUDU-2971 p1: add subprocess module

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/14425 ) Change subject: KUDU-2971 p1: add subprocess module .. Patch Set 2: Code-Review+1 (1 comment) I think this refactoring makes sense. I'm

[kudu-CR] KUDU-2971 p1: add subprocess module

Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14425 ) Change subject: KUDU-2971 p1: add subprocess module .. Patch Set 2: (3 comments)

[kudu-CR] KUDU-2971 p1: add subprocess module

Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14425 ) Change subject: KUDU-2971 p1: add subprocess module .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-2971 p1: add subprocess module

Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14425 ) Change subject: KUDU-2971 p1: add subprocess module .. Patch Set 2: (2 comments) Not a full review, but I'm gathering more context

[kudu-CR] KUDU-2971 p1: add subprocess module

Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/14425 ) Change subject: KUDU-2971 p1: add subprocess module .. Patch Set 2: (4 comments)

[kudu-CR] KUDU-2971 p1: add subprocess module

Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14425 to look at the new patch set (#2). Change subject: KUDU-2971 p1: add subprocess module .. KUDU-2971 p1:

[kudu-CR] KUDU-2971 p1: add subprocess module

Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14425 Change subject: KUDU-2971 p1: add subprocess module .. KUDU-2971 p1: add subprocess module This commit separates a facility class that