New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 748240 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Dec 18
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Feature



Sign in to add a comment

need data pipe to return an error status code when completes

Project Member Reported by qin...@chromium.org, Jul 24 2017

Issue description

Currently data pipe doesn't support aborting with an error status.

We need this feature to avoid extra data copy in DownloadRequestCore.

Currently DownloadRequestCore uses a ByteStream reader/writer pair to make an extra copy the data from http response.
In order to get rid of that copy when data pipe is used, we want to something similar to ByteStreamWrite::Close(status) so that the consumer side can know the error.

I think this feature is also requested by others here:
https://groups.google.com/a/chromium.org/forum/#!topic/chromium-mojo/HPOwhIVrAqc

 

Comment 1 by jam@chromium.org, Jul 24 2017

Cc: roc...@chromium.org jam@chromium.org yzshen@chromium.org
Components: Internals>Mojo
Owner: ----

Comment 2 by roc...@chromium.org, Jul 24 2017

We're unlikely to provide this ability any time soon. It would require changing Mojo's internal IPC protocol in a way that would break binary backwards-compatibility. Is the need really strong enough to justify such an effort? Realistically, any unnecessary reading is bounded by the capacity of the data pipe. How big of a pipe are we talking about?

Comment 3 by qin...@chromium.org, Jul 24 2017

Unnecessary reading is not a big issue here. The problem is passing a status code to the consumer.

Comment 4 by roc...@chromium.org, Jul 24 2017

You should just use a message pipe for that.
Cc: -roc...@chromium.org rockot@google.com
Status: WontFix (was: Untriaged)
No longer relevant.

Sign in to add a comment