Payload registry has to do a linear search to find payloads

[bdraco]

aiohttp/aiohttp/payload.py

Line 97 in 21f5f92

note: we need zope.interface for more efficient adapter search

There is a note that its inefficent.

We might be able to clean it up by giving each payload a key as part of the base class and then doing a dict lookup instead

[Dreamsorcerer]

We might be able to clean it up by giving each payload a key as part of the base class and then doing a dict lookup instead

Doesn't really work, because it works with isinstance checks, which could be a subclass or an abc. I thought about this previously and didn't think of any improvements. Not sure what zope.interface is though, might be worth looking into.

[bdraco]

Maybe if we have a property that is a key thats the same for the subclassed object.. not sure if that will work. Needs some more thought.

[Dreamsorcerer]

Not sure what you mean, the subclasses are e.g. subclass of bytes or (more realistically) a subclass of io.BufferedIO. They are not our classes, just arbitrary types that might be used to store/stream data.

[bdraco]

I need to dig at it a bit more to come up with a solution. This one will take a bit of thought so I opened the issue so I don't forget to come back to it.