StatelessKnowledgeSessionImpl execute with command

[Dimitri Gamkrelidze]

Hello everyone,

When executing StatelessKnowledgeSession with command,  if it's BatchExecutionCommand , it forces the ServiceLoader to load Command service.

Cannot it be instanciated once? Most time is spend on this operation

public <T> T execute(Command<T> command) {
StatefulKnowledgeSession ksession = newWorkingMemory();

RegistryContext context = RegistryContext.create().register( KieSession.class, ksession );

try {
if ( command instanceof BatchExecutionCommand ) {
context.register( ExecutionResults.class, KieServices.get().getCommands().newExecutionResults() );
}

 

You can see same rules flamegraphs , when I made static field and reused the KIE_COMMAND

Thanks

[Mario Fusco]

Hi,

Great finding, thanks a lot for having reported it.

I reproduced and reported the problem here https://github.com/apache/incubator-kie-issues/issues/635 and fixed it on our main branch with this commit https://github.com/apache/incubator-kie-drools/commit/430ea3788c561027cb0ff8b7022b5bad0cbb4f0b

Thanks again,

Mario

--
You received this message because you are subscribed to the Google Groups "Drools Setup" group.
To unsubscribe from this group and stop receiving emails from it, send an email to drools-setup...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/drools-setup/d7738172-da79-4dcc-b530-b5593fa3d11en%40googlegroups.com.

[Dimitri Gamkrelidze]

Thank you Mario.