commit | 062d422cb604f4acbb547b1ab28ac90cf1fd8582 | [log] [tgz] |
---|---|---|
author | Asim Shankar <ashankar@google.com> | Mon Aug 18 11:14:42 2014 -0700 |
committer | Asim Shankar <ashankar@google.com> | Mon Aug 18 11:14:49 2014 -0700 |
tree | 23aba7e65b2aafb9e702df17e7c0dc861b4b4050 | |
parent | 62ab2005d7292ec9c55f1c211e74b818b194f8fc [diff] |
veyron/runtimes/google/ipc: Reduce test flakiness. Some tests were setup to always call ipc.Client.CloseSend and check for the error. However, when there are no streaming arguments, it is possible that the server closes the stream.Flow that is used for the RPC before the CloseSend message is sent, and thus CloseSend will return an error. I suspect that this "race" is the cause for much of the flakiness reported in: https://code.google.com/p/envyor/issues/detail?id=257 Thanks to jsimsa@ for planting the idea for this race being possible (https://veyron-review.googlesource.com/#/c/3934/) Change-Id: I0446d48195d1bd9d8152861ab491d8c4cd40bcc4