edgedb icon indicating copy to clipboard operation
edgedb copied to clipboard

Initial draft of wasm server integration

Open tailhook opened this issue 2 years ago • 4 comments

This is as simple as it can get:

  1. Has single version of extension without any configuration
  2. Relies on wasm server listening on $runstate_dir/.s.wasm_ext
  3. Uses unix socket with one connection per request (similarly to SCGI)
  4. Uses Pythonic Pickle to serialize request/response

Technically it works, but doesn't make too much sense to integrate right now. Anyone having access to edgedb/edgedb-wasm repo (still private) can play with it though.

tailhook avatar Mar 16 '22 08:03 tailhook

Can we add some basic tests here?

elprans avatar May 04 '22 20:05 elprans

This is generally ready, but I'm not sure if we want to fix GC issue somewhere else.

tailhook avatar May 26 '22 12:05 tailhook

This is generally ready, but I'm not sure if we want to fix GC issue somewhere else.

#3904 should've fixed most - if not all - of the GC issue here, so that we don't need to override StreamReaderProtocol in this PR. I'm looking into the create_task()s in uvloop.

fantix avatar May 26 '22 14:05 fantix

Okay, removed that hack.

tailhook avatar May 27 '22 09:05 tailhook