feat: op registration during calls

[afinch7]

prerequisite for #3372

[bartlomieju]

😭 this might impact performance, hot path will require to acquire a read lock everytime

[afinch7]

It doesn't look like it did in benchmarks.

[bartlomieju]

Yes, but they're flaky... Anyway why do you need to put dispatchers behind the lock? In native plugin PR you already have Arc<Mutex<Isolate>> available so you should be able to register new ops quite easily

[afinch7]

In most cases during dispatch we already have a lock on isolate somewhere else.

[bartlomieju]

I see... I'm curious to see it combined with native plugins PR

[ry]

LGTM - I'm worried about the locks too, but I think we need this functionality. Let's keep an eye on the benchmarks after this and see if there are any noticeable regressions.

( and this nice work @afinch7 )

[afinch7]

It doesn't look like it messed with the benchmarks much at all. I think it would take something pretty drastic on the rust side to really be noticeable beyond max latency.