oleg-zinovev commented on PR #12096:
URL: https://github.com/apache/ignite/pull/12096#issuecomment-4555456522
@alex-plekhanov
I’m a bit confused about the logic of
`org.apache.ignite.internal.processors.query.calcite.exec.ExecutionContext#execute`.
I assumed that the entire query execution takes place in the threads of the
shared executor, and that the executor’s implementation ensures the
happens‑before relationship for the fields of
`org.apache.ignite.internal.processors.query.calcite.exec.rel.Node`.
However,
`org.apache.ignite.internal.processors.query.calcite.exec.rel.Inbox#onBatchReceived`
is called from
`org/apache/ignite/internal/processors/query/calcite/exec/ExchangeServiceImpl.java:321`
within `IgniteThread` and does not delegate the execution to
`org.apache.ignite.internal.processors.query.calcite.exec.ExecutionContext#execute`.
Is this correct, and in this case, how is the happens‑before relationship
guaranteed for the fields of
`org.apache.ignite.internal.processors.query.calcite.exec.rel.Node`?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
