api/server: fix the add_stream endpoint
ClosedPublic
Actions

Authored by douardda on Mar 19 2019, 11:44 AM.

Details

Reviewers

Group Reviewers

Commits

rDOBJS039434c41018: api/server: fix the add_stream endpoint
rDOBJSf8de5851560e: api/server: replace asyncio coroutines by native ones

Summary

since received chunks must be decoded one at a time.

also replace asyncio coroutines by native ones

Diff Detail

Repository

rDOBJS Object storage

Branch

master

Lint

Lint Skipped

Unit

Unit Tests Skipped

Build Status

Buildable 4719
Build 6286: tox-on-jenkins	Jenkins
Build 6285: arc lint + arc unit

Event Timeline

douardda created this revision.Mar 19 2019, 11:44 AM

Herald added a reviewer: Reviewers. · View Herald TranscriptMar 19 2019, 11:44 AM

Build is green
See https://jenkins.softwareheritage.org/job/DOBJS/job/tox/22/ for more details.

Harbormaster completed remote builds in B4719: Diff 4003.Mar 19 2019, 11:46 AM

also replace asyncio coroutines by native ones

Aren't these only available starting with python 3.7?

nvm, 3.7 makes them keyword, but this syntax is valid in 3.5 too.

This revision is now accepted and ready to land.Mar 19 2019, 12:08 PM

Closed by commit rDOBJSf8de5851560e: api/server: replace asyncio coroutines by native ones (authored by douardda). · Explain WhyMar 20 2019, 12:17 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

swh/

objstorage/

api/

server.py

91 lines

Diff 4003

View Options

swh/objstorage/api/server.py

api/server: fix the add_stream endpointClosedPublicActions