Ondřej Švanda
Ondřej Švanda
> expose an object that answers some questions about the driver - e.g. "does it implement optional X method This is the best option in my opinion, too. A controlled...
If I understand correctly, a big inspiration for this library was Knex, which supports this by nesting multiple "tx.transaction" calls. When implementing helper methods, we can't be sure if this...
If it is 90% of use-cases, then I think calling it `nestedTransaction` is an unnecessary visual bloat. Especially if nesting `transaction` calls is already an established pattern in other libraries....
@igalklebanov You raise some very good points that I didn't consider. There's way more implicit behavior in nested transactions, like, as you mentioned, transaction settings that _won't_ apply when nested....
What about `inTransaction`? Or `isInTansaction`? And make it a property getter rather than a method.
Hi, as far as I'm concerned, I don't see why it would be an issue. Both OTEL and nestjs-cls use their own isolated instance of AsyncLocalStorage. I have never had...
Another thing that came to my mind is to check your Node version. AsyncLocalStorage was made stable in v16, but there were still some issues with propagation in certain situations...
@pvogel1967 Hi, have you eventually figured this out, or do you need some assistance?
@pvogel1967 Hi, have you moved forward with the issue since, or is it no longer relevant? In that case, I'll go ahead and close the issue.
That's good to hear. Could you at least give a hint in a comment here? Not that I wouldn't appreciate a PR to the docs, but that seems like a...