Shu Ming has posted comments on this change. Change subject: Fix data pass with async execution ......................................................................
Patch Set 1: I would prefer that you didn't submit this The patch subject is confusing to me. It tells me it fixes a problem about syn/async problem. However, in the code, it really change write to p.stdin instead of p.stdout. It looks like fixing a wrong pipe fd problem to me, not a sync/async problem. -- To view, visit http://gerrit.ovirt.org/8081 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ibe496e74a24721ba00c596d36286bcdafdc20069 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Eduardo <[email protected]> Gerrit-Reviewer: Shu Ming <[email protected]> Gerrit-Reviewer: Yeela Kaplan <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
