TriggerReactor

TriggerReactor

24.6k Downloads

Indicate which executors require sync mode

gerzytet opened this issue · 4 comments

commented

The Sync page contains some general guideline so to determine which operations require sync mode, but most people don’t read this page Andnnsn it isn’t always clear which executors require sync.
We should add markers to the executors that require sync mode.

commented

Can you clarify what it means by executors require sync mode?

Executors run only in server thread, so it always syncs regardless of Triggers.

commented

Some executors have to execute with the server thread, like #SETBLOCK, these need to be marked specificaly

commented

@gerzytet
By the design itself, anything runs as Executor or Placeholder runs in server thread.

Code interpretation is async though.

If you are talking about the direct method invocation, then sync mode is required for sure, but Executor and Placeholder does not since they run in server thread anyway.

commented

@wysohn
Didn't know that, oops.