runtimes/google/ipc/...: consistently return io.EOF to avoid spammy log messages.
- vc/writer now returns io.EOF if the flow has been closed remotely.
Change-Id: I5bc5190d7857c949e20f76570545997383224d37
diff --git a/runtimes/google/ipc/stream/vif/vif.go b/runtimes/google/ipc/stream/vif/vif.go
index 6f97362..9a9f025 100644
--- a/runtimes/google/ipc/stream/vif/vif.go
+++ b/runtimes/google/ipc/stream/vif/vif.go
@@ -386,6 +386,10 @@
if vc, _, _ := vif.vcMap.Find(m.VCI); vc != nil {
vif.vcMap.Delete(vc.VCI())
vlog.VI(2).Infof("CloseVC(%+v) on VIF %s", m, vif)
+ // TODO(cnicolaou): it would be nice to have a method on VC
+ // to indicate a 'remote close' rather than a 'local one'. This helps
+ // with error reporting since we expect reads/writes to occur
+ // after a remote close, but not after a local close.
vc.Close(fmt.Sprintf("remote end closed VC(%v)", m.Error))
return nil
}
