fjtirado commented on code in PR #3777:
URL:
https://github.com/apache/incubator-kie-kogito-runtimes/pull/3777#discussion_r1842127359
##########
jbpm/jbpm-flow/src/main/java/org/kie/kogito/process/impl/AbstractProcessInstance.java:
##########
@@ -85,7 +85,7 @@ public abstract class AbstractProcessInstance<T extends
Model> implements Proces
protected final T variables;
protected final AbstractProcess<T> process;
protected InternalProcessRuntime rt;
- protected WorkflowProcessInstance processInstance;
+ protected volatile WorkflowProcessInstance processInstance;
Review Comment:
Probably processInstance field should be embedded into an atomic reference
for complete thread safety, but volatile might have the same effect (its just a
nul check after all) with few code changes
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
